Do not use overlays when VR shell is enabled.

media/blink/webmediaplayer_impl.h

Index: media/blink/webmediaplayer_impl.h
diff --git a/media/blink/webmediaplayer_impl.h b/media/blink/webmediaplayer_impl.h
index 020adcea097348c4a25ddcf5e7fe15aac47540d1..a1e4628655cb72311cb5017b73014844ae564254 100644
--- a/media/blink/webmediaplayer_impl.h
+++ b/media/blink/webmediaplayer_impl.h
@@ -207,6 +207,7 @@ class MEDIA_BLINK_EXPORT WebMediaPlayerImpl
                                     PipelineStatus status);
   gfx::Size GetCanvasSize() const;
   void SetDeviceScaleFactor(float scale_factor);
+  void SetEnableFullscreenOverlays(bool enable_overlays);
   void setPoster(const blink::WebURL& poster) override;
   void SetUseFallbackPath(bool use_fallback_path);
 #endif
@@ -535,9 +536,9 @@ class MEDIA_BLINK_EXPORT WebMediaPlayerImpl
   // Force to use SurfaceView instead of SurfaceTexture on Android.
   bool force_video_overlays_;
 
-  // Prevent use of SurfaceView on Android. (Ignored when
+  // Allow use of SurfaceView on Android. (Ignored when

[ddorwin, 2016/12/15 01:23:52]

s/Allow use of/Use/

The comment is very Android-specific, which is not clear from the variable name
nor is the code Android-specific. Perhaps there should be a general comment with
a separate sentence that says, "On Android, ..."

This "Ignored" relationship is a bit confusing and fragile. It would be nice to
clean that up. See my comments in the .cc file.

[amp, 2016/12/15 21:49:02]

I made an attempt at cleaning up the comments.  Per cc comments I'll leave the
force flag for a later change (but I did update it's comments a bit).

   // |force_video_overlays_| is true.)
-  bool disable_fullscreen_video_overlays_;
+  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