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

Unified Diff: media/blink/webmediaplayer_impl.cc

Issue 2643893002: Skip video overlay surface management when in VR. (Closed)
Patch Set: Removed public API changes and changed references to cast to base instead. Created 3 years, 11 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 | « media/blink/webmediaplayer_impl.h ('k') | testing/variations/fieldtrial_testing_config.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/blink/webmediaplayer_impl.cc
diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc
index aabb04079f5861bd4dd8c600c9dab1a20ae6ada7..d2a7de214077680bb15827851dc48f4b1d691924 100644
--- a/media/blink/webmediaplayer_impl.cc
+++ b/media/blink/webmediaplayer_impl.cc
@@ -253,6 +253,9 @@ WebMediaPlayerImpl::WebMediaPlayerImpl(
force_video_overlays_ = base::CommandLine::ForCurrentProcess()->HasSwitch(
switches::kForceVideoOverlays);
+ enable_fullscreen_video_overlays_ =
+ base::FeatureList::IsEnabled(media::kOverlayFullscreenVideo);
+
if (delegate_) {
delegate_id_ = delegate_->AddObserver(this);
delegate_->SetIdle(delegate_id_, true);
@@ -314,10 +317,6 @@ void WebMediaPlayerImpl::load(LoadType load_type,
DoLoad(load_type, url, cors_mode);
}
-void WebMediaPlayerImpl::SetEnableFullscreenOverlays(bool enable_overlays) {
- enable_fullscreen_video_overlays_ = enable_overlays;
-}
-
bool WebMediaPlayerImpl::supportsOverlayFullscreenVideo() {
#if defined(OS_ANDROID)
return true;
« no previous file with comments | « media/blink/webmediaplayer_impl.h ('k') | testing/variations/fieldtrial_testing_config.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698