Objective C API to read and set RtpParameters

webrtc/api/objc/RTCMediaStreamTrack.mm

 /*
  *  Copyright 2015 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.
  */
 
@@ skipping 29 matching lines @@
 
 - (NSString *)description {
   NSString *readyState = [[self class] stringForState:self.readyState];
   return [NSString stringWithFormat:@"RTCMediaStreamTrack:\n%@\n%@\n%@\n%@",
                                     self.kind,
                                     self.trackId,
                                     self.isEnabled ? @"enabled" : @"disabled",
                                     readyState];
 }
 
+- (BOOL)isEqual:(id)object {
+  if (self == object) {
+    return YES;
+  }
+  if (![object isKindOfClass:[RTCMediaStreamTrack class]]) {

[tkchin_webrtc, 2016/04/08 22:45:46]

[object isMemberOfClass:[self class]]? Do we care about subclasses?

[skvlad, 2016/04/08 23:23:23]

Changed to isMemberOfClass:.

+    return NO;
+  }
+  return [self isEqualToTrack:(RTCMediaStreamTrack *)object];
+}
+
+- (NSUInteger)hash {
+  return (NSUInteger) _nativeTrack.get();

[tkchin_webrtc, 2016/04/08 22:45:46]

nit: remove space
Is this a safe conversion?

[skvlad, 2016/04/08 23:23:23]

It's safe enough for a hash function, even if two different truncated pointers
end up having the same value. Two equal pointers will remain equal after
truncation.

+}
+
 #pragma mark - Private
 
 - (rtc::scoped_refptr<webrtc::MediaStreamTrackInterface>)nativeTrack {
   return _nativeTrack;
 }
 
 - (instancetype)initWithNativeTrack:
     (rtc::scoped_refptr<webrtc::MediaStreamTrackInterface>)nativeTrack
                                type:(RTCMediaStreamTrackType)type {
   NSParameterAssert(nativeTrack);
   if (self = [super init]) {
     _nativeTrack = nativeTrack;
     _type = type;
   }
   return self;
 }
 
+- (instancetype)initWithNativeTrack:
+    (rtc::scoped_refptr<webrtc::MediaStreamTrackInterface>)nativeTrack {
+  NSParameterAssert(nativeTrack);
+  if (nativeTrack->kind() ==
+      std::string(webrtc::MediaStreamTrackInterface::kAudioKind)) {
+    return [self initWithNativeTrack:nativeTrack
+                                type:RTCMediaStreamTrackTypeAudio];
+  }
+  if (nativeTrack->kind() ==
+      std::string(webrtc::MediaStreamTrackInterface::kVideoKind)) {
+    return [self initWithNativeTrack:nativeTrack
+                                type:RTCMediaStreamTrackTypeVideo];
+  }
+  return nil;
+}
+
+- (BOOL)isEqualToTrack:(RTCMediaStreamTrack *)track {
+  if (!track) {
+    return NO;
+  }
+  return _nativeTrack == track->_nativeTrack;

[tkchin_webrtc, 2016/04/08 22:45:46]

nit: don't use -> if possible
just access the nativeTrack property
also, I assume it's safe to do == on refptr like that?

[skvlad, 2016/04/08 23:23:23]

Changed to use the property. refptr has operator== overridden to compare the
underlying raw pointers, so it will be doing the right thing here.

+}
+
 + (webrtc::MediaStreamTrackInterface::TrackState)nativeTrackStateForState:
     (RTCMediaStreamTrackState)state {
   switch (state) {
     case RTCMediaStreamTrackStateLive:
       return webrtc::MediaStreamTrackInterface::kLive;
     case RTCMediaStreamTrackStateEnded:
       return webrtc::MediaStreamTrackInterface::kEnded;
   }
 }
 
@@ skipping 10 matching lines @@
 + (NSString *)stringForState:(RTCMediaStreamTrackState)state {
   switch (state) {
     case RTCMediaStreamTrackStateLive:
       return @"Live";
     case RTCMediaStreamTrackStateEnded:
       return @"Ended";
   }
 }
 
 @end