commit 7f8b4340098a40a9c927cbb16fe3f46a6277fee6
author Harald Alvestrand <hta@webrtc.org> Mon Aug 17 12:57:30 2020 +0200
committer Commit Bot <commit-bot@chromium.org> Mon Aug 17 11:55:55 2020 +0000
tree a6634d927a286038a4739a25498bb49ef35e6d45
parent 16dfcdbc1cd900a9c3d0ebfa294e8e81e17d4200

Modify Android API to use SetDirectionWithError

This clears the decks for deprecating and eventually removing
the nonstandard SetDirection method.

Bug: chromium:980879
Change-Id: Ibc291de3db690e9ef4e6cb3550390d7728f02a83
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/181860
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31948}

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

diff --git a/sdk/android/api/org/webrtc/RtpTransceiver.java b/sdk/android/api/org/webrtc/RtpTransceiver.java
index 021cc90..aff1bfb 100644
--- a/sdk/android/api/org/webrtc/RtpTransceiver.java
+++ b/sdk/android/api/org/webrtc/RtpTransceiver.java
@@ -200,9 +200,9 @@
    * sendrecv, sendonly, recvonly, or inactive.
    * https://w3c.github.io/webrtc-pc/#dom-rtcrtptransceiver-direction
    */
-  public void setDirection(RtpTransceiverDirection rtpTransceiverDirection) {
+  public boolean setDirection(RtpTransceiverDirection rtpTransceiverDirection) {
     checkRtpTransceiverExists();
-    nativeSetDirection(nativeRtpTransceiver, rtpTransceiverDirection);
+    return nativeSetDirection(nativeRtpTransceiver, rtpTransceiverDirection);
   }
 
   /**
@@ -260,6 +260,6 @@
   private static native RtpTransceiverDirection nativeCurrentDirection(long rtpTransceiver);
   private static native void nativeStopInternal(long rtpTransceiver);
   private static native void nativeStopStandard(long rtpTransceiver);
-  private static native void nativeSetDirection(
+  private static native boolean nativeSetDirection(
       long rtpTransceiver, RtpTransceiverDirection rtpTransceiverDirection);
 }