Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(292)

Unified Diff: media/base/android/media_source_player.cc

Issue 1110833004: Move audio focus control from media/ to content/ and make it per WebContents. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments and compile Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/android/media_source_player.cc
diff --git a/media/base/android/media_source_player.cc b/media/base/android/media_source_player.cc
index 7e8f49a4613c4a6c3189ea49ec4c94024688a249..460fbec8fb6ced9c158d034b403ee382df4c37e7 100644
--- a/media/base/android/media_source_player.cc
+++ b/media/base/android/media_source_player.cc
@@ -217,6 +217,11 @@ void MediaSourcePlayer::StartInternal() {
if (pending_event_ != NO_EVENT_PENDING)
return;
+ if (!manager()->RequestPlay(player_id())) {
+ Pause(true);
qinmin 2015/05/23 18:16:06 i think this should be false, the value is use by
mlamouri (slow - plz ping) 2015/05/23 20:05:27 This question might sound silly but what's the lin
+ return;
+ }
+
// When we start, we could have new demuxed data coming in. This new data
// could be clear (not encrypted) or encrypted with different keys. So key
// related info should all be cleared.

Powered by Google App Engine
This is Rietveld 408576698