diff options
author | wjiang <wjiang@codeaurora.org> | 2014-07-11 14:51:00 +0800 |
---|---|---|
committer | emancebo <emancebo@cyngn.com> | 2014-09-04 10:40:21 -0700 |
commit | b64867ee86bc3991d5b07932575b883af06f51bc (patch) | |
tree | 09629537090905159c430627fcf34000d459a15b | |
parent | 812b1e34a4652a3ff1b9bb2c4e617c38078bd1cd (diff) | |
download | android_packages_apps_Gallery2-b64867ee86bc3991d5b07932575b883af06f51bc.tar.gz android_packages_apps_Gallery2-b64867ee86bc3991d5b07932575b883af06f51bc.tar.bz2 android_packages_apps_Gallery2-b64867ee86bc3991d5b07932575b883af06f51bc.zip |
Gallery2: don't pause background music perminently
Gallery app requests transient audio focus twice and explicitly pauses
background music playback. To allow music resume after movie playback
exists, remove redundant audio focus request and pause broadcast.
CRs-Fixed: 695669
Change-Id: I46b88defc887d3e5b5c97acdc4f5a0a11d486a76
-rw-r--r-- | src/com/android/gallery3d/app/MoviePlayer.java | 12 | ||||
-rwxr-xr-x | src/org/codeaurora/gallery3d/video/CodeauroraVideoView.java | 6 |
2 files changed, 1 insertions, 17 deletions
diff --git a/src/com/android/gallery3d/app/MoviePlayer.java b/src/com/android/gallery3d/app/MoviePlayer.java index 54f852448..1a4ea7f28 100644 --- a/src/com/android/gallery3d/app/MoviePlayer.java +++ b/src/com/android/gallery3d/app/MoviePlayer.java @@ -91,11 +91,6 @@ public class MoviePlayer implements private static final int KEYCODE_MEDIA_PLAY = 126; private static final int KEYCODE_MEDIA_PAUSE = 127; - // Copied from MediaPlaybackService in the Music Player app. - private static final String SERVICECMD = "com.android.music.musicservicecommand"; - private static final String CMDNAME = "command"; - private static final String CMDPAUSE = "pause"; - private static final String KEY_VIDEO_CAN_SEEK = "video_can_seek"; private static final String KEY_VIDEO_CAN_PAUSE = "video_can_pause"; private static final String KEY_VIDEO_LAST_DURATION = "video_last_duration"; @@ -294,10 +289,6 @@ public class MoviePlayer implements filter.addAction(Intent.ACTION_SHUTDOWN); mContext.registerReceiver(mReceiver, filter); - Intent i = new Intent(SERVICECMD); - i.putExtra(CMDNAME, CMDPAUSE); - movieActivity.sendBroadcast(i); - if (savedInstance != null) { // this is a resumed activity mVideoPosition = savedInstance.getInt(KEY_VIDEO_POSITION, 0); onRestoreInstanceState(savedInstance); @@ -596,9 +587,6 @@ public class MoviePlayer implements } private void doStartVideo(boolean enableFasten, int position, int duration) { - ((AudioManager) mContext.getSystemService(Context.AUDIO_SERVICE)).requestAudioFocus( - null, AudioManager.STREAM_MUSIC, - AudioManager.AUDIOFOCUS_GAIN_TRANSIENT); doStartVideo(enableFasten, position, duration, true); } diff --git a/src/org/codeaurora/gallery3d/video/CodeauroraVideoView.java b/src/org/codeaurora/gallery3d/video/CodeauroraVideoView.java index dc57b7058..d2b6790ef 100755 --- a/src/org/codeaurora/gallery3d/video/CodeauroraVideoView.java +++ b/src/org/codeaurora/gallery3d/video/CodeauroraVideoView.java @@ -437,11 +437,7 @@ public class CodeauroraVideoView extends SurfaceView implements MediaPlayerContr // not ready for playback just yet, will try again later return; } - // Tell the music playback service to pause - // TODO: these constants need to be published somewhere in the framework. - Intent i = new Intent("com.android.music.musicservicecommand"); - i.putExtra("command", "pause"); - mContext.sendBroadcast(i); + // we shouldn't clear the target state, because somebody might have // called start() previously release(false); |