summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwjiang <wjiang@codeaurora.org>2014-07-11 14:51:00 +0800
committeremancebo <emancebo@cyngn.com>2014-09-04 10:40:21 -0700
commitb64867ee86bc3991d5b07932575b883af06f51bc (patch)
tree09629537090905159c430627fcf34000d459a15b
parent812b1e34a4652a3ff1b9bb2c4e617c38078bd1cd (diff)
downloadandroid_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.java12
-rwxr-xr-xsrc/org/codeaurora/gallery3d/video/CodeauroraVideoView.java6
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);