Bug 1347648 - part3 : only request audio focus from gecko. draft
authorAlastor Wu <alwu@mozilla.com>
Mon, 27 Mar 2017 10:59:10 +0800
changeset 551571 c857af062969040b9bc604ff13b3b798859c5fc3
parent 551570 747299f7bb7d75573276b3438fb906ac364d4510
child 551572 6e921a8dc1f78d18f00342efd1321947d4e3f479
push id51086
push useralwu@mozilla.com
push dateMon, 27 Mar 2017 02:59:50 +0000
bugs1347648
milestone55.0a1
Bug 1347648 - part3 : only request audio focus from gecko. Media control would also be displayed for non-audible media, we shouldn't arbitrary request audio focus . Only request audio focus from gecko which know whether the media is audible. MozReview-Commit-ID: Ke9DCYd0Qh2
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
@@ -308,27 +308,24 @@ public class MediaControlService extends
             }
 
             @Override
             public void onPlay() {
                 Log.d(LOGTAG, "Controller, onPlay");
                 super.onPlay();
                 setState(State.PLAYING);
                 notifyObservers("MediaControl", "resumeMedia");
-                // To make sure we always own audio focus during playing.
-                AudioFocusAgent.notifyStartedPlaying();
             }
 
             @Override
             public void onPause() {
                 Log.d(LOGTAG, "Controller, onPause");
                 super.onPause();
                 setState(State.PAUSED);
                 notifyObservers("MediaControl", "mediaControlPaused");
-                AudioFocusAgent.notifyStoppedPlaying();
             }
 
             @Override
             public void onStop() {
                 Log.d(LOGTAG, "Controller, onStop");
                 super.onStop();
                 setState(State.STOPPED);
                 notifyObservers("MediaControl", "mediaControlStopped");