Add the OnAddTrack callback for Objective-C wrapper.

webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnection.h

 /*
  *  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 88 matching lines @@
     didGenerateIceCandidate:(RTCIceCandidate *)candidate;
 
 /** Called when a group of local Ice candidates have been removed. */
 - (void)peerConnection:(RTCPeerConnection *)peerConnection
     didRemoveIceCandidates:(NSArray<RTCIceCandidate *> *)candidates;
 
 /** New data channel has been opened. */
 - (void)peerConnection:(RTCPeerConnection *)peerConnection
     didOpenDataChannel:(RTCDataChannel *)dataChannel;
 
+@optional
+/** Called when a new track is signaled from remote peer. */
+- (void)peerConnection:(RTCPeerConnection*)peerConnection
+    didAddTrack:(RTCRtpReceiver*)rtpReceiver

[tkchin_webrtc, 2016/11/22 20:41:34]

align :

space after *
(RTCPeerConnection *)
NSArray<RTCMediaStream *> *)

[Zhi Huang, 2016/11/22 23:38:42]

Done.

Should I align the ":" in previous functions such as
"didChangeIceConnectionState" ?

+    attachedStreams:(NSArray<RTCMediaStream*>*)mediaStreams;

[tkchin_webrtc, 2016/11/22 20:41:34]

Seems different than spec?
https://www.w3.org/TR/webrtc/#dom-rtctrackevent

Would've expected:
- (void)peerConnection:(RTCPeerConnection *)peerConnection
didAddTrack:(RTCMediaStreamTrack *)track
streams:(NSArray<RTCMediaStream *> *)streams
transceiver:(RTCRtpTransceiver *)transceiver;

[Zhi Huang, 2016/11/22 23:38:42]

According to the spec, the Receiver is required. We can easily get the Track
from the Receiver, not the other way around.
We don't have transceivers yet.

I think it would be fine as long as the native api provide enough info for JS
api.

I agree that this may seems a little bit confusing. Maybe make it clear with
comments?

+
 @end
 
 RTC_EXPORT
 @interface RTCPeerConnection : NSObject
 
 /** The object that will be notifed about events such as state changes and
  *  streams being added or removed.
  */
 @property(nonatomic, weak, nullable) id<RTCPeerConnectionDelegate> delegate;
 @property(nonatomic, readonly) NSArray<RTCMediaStream *> *localStreams;
@@ skipping 96 matching lines @@
  */
 - (void)statsForTrack:
     (nullable RTCMediaStreamTrack *)mediaStreamTrack
      statsOutputLevel:(RTCStatsOutputLevel)statsOutputLevel
     completionHandler:
     (nullable void (^)(NSArray<RTCLegacyStatsReport *> *stats))completionHandler;
 
 @end
 
 NS_ASSUME_NONNULL_END