Skip video overlay surface management when in VR.

content/public/browser/render_widget_host_view.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RENDER_WIDGET_HOST_VIEW_H_
 #define CONTENT_PUBLIC_BROWSER_RENDER_WIDGET_HOST_VIEW_H_
 
 #include <memory>
 
 #include "base/strings/string16.h"
@@ skipping 49 matching lines @@
   // Tells the View to size and move itself to the specified size and point in
   // screen space.
   virtual void SetBounds(const gfx::Rect& rect) = 0;
 
   // Retrieves the last known scroll position.
   virtual gfx::Vector2dF GetLastScrollOffset() const = 0;
 
   // 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
   // properly transformed; however, coordinates received from an out-of-process
   // iframe renderer process require transformation.
   virtual gfx::Point TransformPointToRootCoordSpace(
       const gfx::Point& point) = 0;
 
   // A floating point variant of the above. PointF values will be snapped to
   // integral points before transformation.
@@ skipping 112 matching lines @@
   // Returns |true| if text is currently being spoken by Mac OS X.
   virtual bool IsSpeaking() const = 0;
   // Stops speaking, if it is currently in progress.
   virtual void StopSpeaking() = 0;
 #endif  // defined(OS_MACOSX)
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RENDER_WIDGET_HOST_VIEW_H_