Bug 1313559 - Don't need to notify gecko if the media was resumed-from-page. draft
authorAlastor Wu <alwu@mozilla.com>
Fri, 28 Oct 2016 11:49:22 +0800
changeset 430771 af147eaa6f32ece6790e781766641894565c19b3
parent 430359 9888f1a23001fde6435e1a9ed7e6d3af8dd988d8
child 535255 33ea69a7f55fbdcc34e6a57cf757cb4743775467
push id33881
push useralwu@mozilla.com
push dateFri, 28 Oct 2016 03:49:59 +0000
bugs1313559
milestone52.0a1
Bug 1313559 - Don't need to notify gecko if the media was resumed-from-page. MozReview-Commit-ID: Cemch546mcS
mobile/android/base/java/org/mozilla/gecko/media/MediaControlService.java
--- a/mobile/android/base/java/org/mozilla/gecko/media/MediaControlService.java
+++ b/mobile/android/base/java/org/mozilla/gecko/media/MediaControlService.java
@@ -108,17 +108,17 @@ public class MediaControlService extends
                     notifyControlInterfaceChanged(ACTION_STOP);
                     mTabReference = new WeakReference<>(null);
                 }
                 break;
             case MEDIA_PLAYING_RESUME:
                 // user resume the paused-by-control media from page so that we
                 // should make the control interface consistent.
                 if (playingTab == tab && !isMediaPlaying()) {
-                    mController.getTransportControls().play();
+                    notifyControlInterfaceChanged(ACTION_PAUSE);
                 }
                 break;
             case CLOSED:
                 if (playingTab == null || playingTab == tab) {
                     // Remove the controls when the playing tab disappeared or was closed.
                     notifyControlInterfaceChanged(ACTION_STOP);
                 }
                 break;