Bug 1332177 - handle ICodec.configure() failure. r?jchen draft
authorJohn Lin <jolin@mozilla.com>
Thu, 19 Jan 2017 16:11:01 +0800
changeset 463489 09424d059073e44cab0e11217d0c860648ef2674
parent 463342 96cb95af530477edb66ae48d98c18533476e57bb
child 542694 e28e0896b95b11f022e8027f05cd9dc3ee1c22ca
push id42085
push userbmo:jolin@mozilla.com
push dateThu, 19 Jan 2017 08:13:03 +0000
reviewersjchen
bugs1332177
milestone53.0a1
Bug 1332177 - handle ICodec.configure() failure. r?jchen MozReview-Commit-ID: KPhI04EA9n3
mobile/android/base/java/org/mozilla/gecko/media/CodecProxy.java
--- a/mobile/android/base/java/org/mozilla/gecko/media/CodecProxy.java
+++ b/mobile/android/base/java/org/mozilla/gecko/media/CodecProxy.java
@@ -119,17 +119,19 @@ public final class CodecProxy {
         mOutputSurface = surface;
         mRemoteDrmStubId = drmStubId;
         mCallbacks = new CallbacksForwarder(callbacks);
     }
 
     boolean init(ICodec remote) {
         try {
             remote.setCallbacks(mCallbacks);
-            remote.configure(mFormat, mOutputSurface, 0, mRemoteDrmStubId);
+            if (!remote.configure(mFormat, mOutputSurface, 0, mRemoteDrmStubId)) {
+                return false;
+            }
             remote.start();
         } catch (RemoteException e) {
             e.printStackTrace();
             return false;
         }
 
         mRemote = remote;
         return true;