Skip video overlay surface management when in VR.

content/browser/media/android/browser_surface_view_manager.cc

Index: content/browser/media/android/browser_surface_view_manager.cc
diff --git a/content/browser/media/android/browser_surface_view_manager.cc b/content/browser/media/android/browser_surface_view_manager.cc
index 4a73cd2f86c8a0d67a01b0c907fa8157d045dd46..3af37f9446f33d579710eda42d99b2c81eee7597 100644
--- a/content/browser/media/android/browser_surface_view_manager.cc
+++ b/content/browser/media/android/browser_surface_view_manager.cc
@@ -63,6 +63,13 @@ void BrowserSurfaceViewManager::DidExitFullscreen(bool release_media_player) {
 
 void BrowserSurfaceViewManager::OnCreateFullscreenSurface(
     const gfx::Size& video_natural_size) {
+  // If we are in virtual reality, no surface view is needed so just return.
+  // TODO(http://crbug.com/673886): Support overlay surfaces in VR using GVR
+  // reprojection video surface.
+  if (render_frame_host_->GetView()->IsInVR()) {

[liberato (no reviews please), 2017/01/18 18:21:48]

one migh tneed SendSurfaceID(kNoSurfaceID), else WMPI will never find out that
the request failed.

[amp, 2017/01/18 19:30:25]

Done.

Nice catch, although it seems to work fine without it...

Line 95 below also seems to be case where it can return without sending back an
id.

+    return;
+  }
+
   // It's valid to get this call if we already own the fullscreen view. We just
   // return the existing surface id.
   if (content_video_view_) {