Skip video overlay surface management when in VR.

content/public/browser/render_widget_host_view.h

Index: content/public/browser/render_widget_host_view.h
diff --git a/content/public/browser/render_widget_host_view.h b/content/public/browser/render_widget_host_view.h
index 71c693bf7c3c09c90b6df776d1afe9dff015fbf2..acff760e61672b70d2d9442c6fc10e2284779008 100644
--- a/content/public/browser/render_widget_host_view.h
+++ b/content/public/browser/render_widget_host_view.h
@@ -67,6 +67,9 @@ class CONTENT_EXPORT RenderWidgetHostView {
   // Sets a flag that indicates if it is in virtual reality mode.
   virtual void SetIsInVR(bool is_in_vr) = 0;
 
+  // Returns true if the current view is in virtual reality mode.
+  virtual bool IsInVR() const = 0;

[Charlie Reis, 2017/01/18 22:34:41]

We can't add this to the public API unless there are callers outside content:

https://www.chromium.org/developers/content-module/content-api
"only expose methods in the public API that embedders need. If a method is only
used by other code in content, it belongs in foo_impl.h and not foo.h."

It looks like it's only being used from
BrowserSurfaceViewManager::OnCreateFullscreenSurface inside content.  Can we
declare it on RenderWidgetHostViewBase instead of RenderWidgetHostView and then
cast at the call site?

(I would have recommended changing RenderFrameHostImpl::GetView() to return a
RenderWidgetHostViewBase, since it's a covariant return type for
RenderFrameHostImpl::GetView().  It looks like
BrowserSurfaceViewManager::render_frame_host_ isn't a RenderFrameHostImpl,
though.)

+
   // Coordinate points received from a renderer process need to be transformed
   // to the top-level frame's coordinate space. For coordinates received from
   // the top-level frame's renderer this is a no-op as they are already