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

Unified Diff: content/renderer/media/webmediaplayer_ms.cc

Issue 2428353005: Tuning WebRTC audio focus type (Closed)
Patch Set: fixed nits Created 4 years, 2 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
« no previous file with comments | « content/browser/media/session/media_session.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/webmediaplayer_ms.cc
diff --git a/content/renderer/media/webmediaplayer_ms.cc b/content/renderer/media/webmediaplayer_ms.cc
index ecb27e684afc18d5af4752ee921830e7c0ca17ac..e0eb7edcb4b871f7b19b696369479d397f124d59 100644
--- a/content/renderer/media/webmediaplayer_ms.cc
+++ b/content/renderer/media/webmediaplayer_ms.cc
@@ -181,12 +181,16 @@ void WebMediaPlayerMS::play() {
audio_renderer_->Play();
if (delegate_) {
- // TODO(perkj, magjed): We send a duration of 1 second here to avoid
- // creating an interactive media session on Android. We'd like to use zero
- // here, but that is treated as an unknown duration and assumed to be
- // interactive. See http://crbug.com/595297 for more details.
+ // TODO(perkj, magjed): We use Uncontrollable type here to avoid creating an
+ // interactive media session on Android. See http://crbug.com/596516 for
+ // more details.
+#if defined(OS_ANDROID)
delegate_->DidPlay(delegate_id_, hasVideo(), hasAudio(), false,
media::MediaContentType::Uncontrollable);
+#else // defined(OS_ANDROID)
+ delegate_->DidPlay(delegate_id_, hasVideo(), hasAudio(), false,
+ media::MediaContentType::Persistent);
+#endif // defined(OS_ANDROID)
}
paused_ = false;
@@ -446,12 +450,11 @@ void WebMediaPlayerMS::OnPlay() {
}
void WebMediaPlayerMS::OnPause() {
- // TODO(perkj, magjed): See TODO in Onplay().
+ // TODO(perkj, magjed): See TODO in OnPlay().
}
void WebMediaPlayerMS::OnVolumeMultiplierUpdate(double multiplier) {
- volume_multiplier_ = multiplier;
- setVolume(volume_);
+ // TODO(perkj, magjed): See TODO in OnPlay().
}
bool WebMediaPlayerMS::copyVideoTextureToPlatformTexture(
« no previous file with comments | « content/browser/media/session/media_session.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698