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

Unified Diff: media/blink/webmediaplayer_impl.h

Issue 2439543003: Do not use overlays when VR shell is enabled. (Closed)
Patch Set: address final comments (and a rebase) Created 4 years 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/renderer/render_frame_impl.cc ('k') | media/blink/webmediaplayer_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/blink/webmediaplayer_impl.h
diff --git a/media/blink/webmediaplayer_impl.h b/media/blink/webmediaplayer_impl.h
index 761dbc7a880fbcb99a99b1f58551d49e8fa81d36..711799d54c3e64fc2c962927abbc6231eaad651c 100644
--- a/media/blink/webmediaplayer_impl.h
+++ b/media/blink/webmediaplayer_impl.h
@@ -178,6 +178,7 @@ class MEDIA_BLINK_EXPORT WebMediaPlayerImpl
blink::WebContentDecryptionModule* cdm,
blink::WebContentDecryptionModuleResult result) override;
+ void SetEnableFullscreenOverlays(bool enable_overlays);
bool supportsOverlayFullscreenVideo() override;
void enteredFullscreen() override;
void exitedFullscreen() override;
@@ -536,12 +537,15 @@ class MEDIA_BLINK_EXPORT WebMediaPlayerImpl
// cleared by the decoder.
SurfaceCreatedCB set_surface_cb_;
- // Force to use SurfaceView instead of SurfaceTexture on Android.
+ // On Android an overlay surface means using
+ // SurfaceView instead of SurfaceTexture.
+
+ // Use overlays for all video.
bool force_video_overlays_;
- // Prevent use of SurfaceView on Android. (Ignored when
- // |force_video_overlays_| is true.)
- bool disable_fullscreen_video_overlays_;
+ // Use overlays for fullscreen video.
+ // (Implied if |force_video_overlays_| is true.)
+ bool enable_fullscreen_video_overlays_;
// Suppresses calls to OnPipelineError() after destruction / shutdown has been
// started; prevents us from spuriously logging errors that are transient or
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | media/blink/webmediaplayer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698