Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(466)

Unified Diff: webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCVideoEncoderSettings.mm

Issue 2977213002: Reland of Injectable Obj-C video codecs (Closed)
Patch Set: Add checks to make sure destroy is called Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCVideoEncoderSettings.mm
diff --git a/webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCVideoEncoderSettings.mm b/webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCVideoEncoderSettings.mm
new file mode 100644
index 0000000000000000000000000000000000000000..3ddf63bb7b2c14fd57356c685f08868879f2813c
--- /dev/null
+++ b/webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCVideoEncoderSettings.mm
@@ -0,0 +1,72 @@
+/*
+ * Copyright 2017 The WebRTC project authors. All Rights Reserved.
+ *
+ * Use of this source code is governed by a BSD-style license
+ * that can be found in the LICENSE file in the root of the source
+ * tree. An additional intellectual property rights grant can be found
+ * in the file PATENTS. All contributing project authors may
+ * be found in the AUTHORS file in the root of the source tree.
+ */
+
+#import "WebRTC/RTCVideoCodec.h"
+
+#import "NSString+StdString.h"
+#import "RTCVideoCodec+Private.h"
+#import "WebRTC/RTCVideoCodecFactory.h"
+
+@implementation RTCVideoEncoderSettings
+
+@synthesize name = _name;
+@synthesize width = _width;
+@synthesize height = _height;
+@synthesize startBitrate = _startBitrate;
+@synthesize maxBitrate = _maxBitrate;
+@synthesize minBitrate = _minBitrate;
+@synthesize targetBitrate = _targetBitrate;
+@synthesize maxFramerate = _maxFramerate;
+@synthesize qpMax = _qpMax;
+
+- (instancetype)initWithVideoCodec:(const webrtc::VideoCodec *__nullable)videoCodec {
Chuck 2017/07/21 15:59:48 initWithNativeVideoCode, no __nullable
+ if (self = [super init]) {
+ if (videoCodec) {
+ rtc::Optional<const char *> codecName = CodecTypeToPayloadName(videoCodec->codecType);
+ if (codecName) {
+ _name = [NSString stringWithUTF8String:codecName.value()];
+ }
+
+ _width = videoCodec->width;
+ _height = videoCodec->height;
+ _startBitrate = videoCodec->startBitrate;
+ _maxBitrate = videoCodec->maxBitrate;
+ _minBitrate = videoCodec->minBitrate;
+ _targetBitrate = videoCodec->targetBitrate;
+ _maxFramerate = videoCodec->maxFramerate;
+ _qpMax = videoCodec->qpMax;
+ }
+ }
+
+ return self;
+}
+
+- (std::unique_ptr<webrtc::VideoCodec>)toCpp {
Chuck 2017/07/21 15:59:48 createNativeVideoCodec
+ auto codecSettings = std::unique_ptr<webrtc::VideoCodec>(new webrtc::VideoCodec);
+
+ rtc::Optional<webrtc::VideoCodecType> codecType =
+ webrtc::PayloadNameToCodecType([NSString stdStringForString:_name]);
+ if (codecType) {
+ codecSettings->codecType = codecType.value();
+ }
+
+ codecSettings->width = _width;
+ codecSettings->height = _height;
+ codecSettings->startBitrate = _startBitrate;
+ codecSettings->maxBitrate = _maxBitrate;
+ codecSettings->minBitrate = _minBitrate;
+ codecSettings->targetBitrate = _targetBitrate;
+ codecSettings->maxFramerate = _maxFramerate;
+ codecSettings->qpMax = _qpMax;
+
+ return codecSettings;
+}
+
+@end

Powered by Google App Engine
This is Rietveld 408576698