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

Side by Side Diff: webrtc/api/objc/RTCRtpParameters.mm

Issue 1885473004: Adding codecs to the RtpParameters returned by an RtpSender. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fixing minor issues Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright 2016 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 #import "RTCRtpParameters+Private.h" 11 #import "RTCRtpParameters+Private.h"
tkchin_webrtc 2016/04/14 17:52:51 ditto add a blank line after interface import
Taylor Brandstetter 2016/04/14 18:17:22 Done.
12 #import "RTCRtpCodecParameters+Private.h"
12 #import "RTCRtpEncodingParameters+Private.h" 13 #import "RTCRtpEncodingParameters+Private.h"
13 14
14 @implementation RTCRtpParameters 15 @implementation RTCRtpParameters
15 16
16 @synthesize encodings = _encodings; 17 @synthesize encodings = _encodings;
18 @synthesize codecs = _codecs;
17 19
18 - (instancetype)init { 20 - (instancetype)init {
19 return [super init]; 21 return [super init];
20 } 22 }
21 23
22 - (instancetype)initWithNativeParameters: 24 - (instancetype)initWithNativeParameters:
23 (const webrtc::RtpParameters &)nativeParameters { 25 (const webrtc::RtpParameters &)nativeParameters {
24 if (self = [self init]) { 26 if (self = [self init]) {
25 NSMutableArray *encodings = [[NSMutableArray alloc] init]; 27 NSMutableArray *encodings = [[NSMutableArray alloc] init];
26 for (const auto &encoding : nativeParameters.encodings) { 28 for (const auto &encoding : nativeParameters.encodings) {
27 [encodings addObject:[[RTCRtpEncodingParameters alloc] 29 [encodings addObject:[[RTCRtpEncodingParameters alloc]
28 initWithNativeParameters:encoding]]; 30 initWithNativeParameters:encoding]];
29 } 31 }
30 _encodings = encodings; 32 _encodings = encodings;
33
34 NSMutableArray *codecs = [[NSMutableArray alloc] init];
35 for (const auto &codec : nativeParameters.codecs) {
36 [codecs addObject:[[RTCRtpCodecParameters alloc]
37 initWithNativeParameters:codec]];
38 }
39 _codecs = codecs;
31 } 40 }
32 return self; 41 return self;
33 } 42 }
34 43
35 - (webrtc::RtpParameters)nativeParameters { 44 - (webrtc::RtpParameters)nativeParameters {
36 webrtc::RtpParameters parameters; 45 webrtc::RtpParameters parameters;
37 for (RTCRtpEncodingParameters *encoding in _encodings) { 46 for (RTCRtpEncodingParameters *encoding in _encodings) {
38 parameters.encodings.push_back(encoding.nativeParameters); 47 parameters.encodings.push_back(encoding.nativeParameters);
39 } 48 }
49 for (RTCRtpCodecParameters *codec in _codecs) {
50 parameters.codecs.push_back(codec.nativeParameters);
51 }
40 return parameters; 52 return parameters;
41 } 53 }
42 54
43 @end 55 @end
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698