Add PeerConnectionObserver#onRemoveTrack to android sdk

webrtc/sdk/android/api/org/webrtc/PeerConnection.java

 /*
  *  Copyright 2013 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 73 matching lines @@
     /** Triggered when a remote peer opens a DataChannel. */
     public void onDataChannel(DataChannel dataChannel);
 
     /** Triggered when renegotiation is necessary. */
     public void onRenegotiationNeeded();
 
     /**
      * Triggered when a new track is signaled by the remote peer, as a result of
      * setRemoteDescription.
      */
-    public void onAddTrack(RtpReceiver receiver, MediaStream[] mediaStreams);

[Taylor Brandstetter, 2017/08/29 19:48:09]

It's not safe to change the signature of this; existing applications may be
depending on it.

[korniltsev, 2017/08/30 21:47:47]

reverted it back

+    public void onAddTrack(MediaStreamTrack track, MediaStream[] mediaStreams);
+
+    /**
+     * Triggered when a previously added remote track is removed by the remote peer,
+     * as a result of setRemoteDescription.
+     */
+    void onRemoveTrack(MediaStreamTrack track, MediaStream[] mediaStreams);

[Taylor Brandstetter, 2017/08/29 19:48:09]

Should be public. Also, can you give it an empty default implementation so that
this change doesn't break existing Observer subclasses? I believe this is
possible now, since the chromium build infrastructure recently started
supporting Java 8 features.

[korniltsev, 2017/08/30 21:47:47]

Done.

   }
 
   /** Java version of PeerConnectionInterface.IceServer. */
   public static class IceServer {
     // List of URIs associated with this server. Valid formats are described
     // in RFC7064 and RFC7065, and more may be added in the future. The "host"
     // part of the URI may contain either an IP address or a hostname.
     public final String uri;
     public final String username;
     public final String password;
@@ skipping 326 matching lines @@
   private native RtpSender nativeCreateSender(String kind, String stream_id);
 
   private native List<RtpSender> nativeGetSenders();
 
   private native List<RtpReceiver> nativeGetReceivers();
 
   private native boolean nativeStartRtcEventLog(int file_descriptor, int max_size_bytes);
 
   private native void nativeStopRtcEventLog();
 }