Stop using the mediaFullscreenRequiresUserGesture setting

content/browser/renderer_host/render_view_host_impl.cc

Index: content/browser/renderer_host/render_view_host_impl.cc
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc
index 17c6167e9f3096324361df7b5d47d37d9896de3f..8fcb988ba38e120fa9c7e4873f59b4afda7b09a9 100644
--- a/content/browser/renderer_host/render_view_host_impl.cc
+++ b/content/browser/renderer_host/render_view_host_impl.cc
@@ -450,8 +450,6 @@ WebPreferences RenderViewHostImpl::GetWebkitPrefs(const GURL& url) {
 #if defined(OS_ANDROID)
   prefs.user_gesture_required_for_media_playback = !command_line.HasSwitch(
       switches::kDisableGestureRequirementForMediaPlayback);
-  prefs.user_gesture_required_for_media_fullscreen = !command_line.HasSwitch(
-      switches::kDisableGestureRequirementForMediaFullscreen);

[acolwell GONE FROM CHROMIUM, 2014/03/26 17:47:31]

Is the plan to remove the kDisableGestureRequirementForMediaFullscreen decl in a
follow-up CL? It seems like the only other reference is in
browser_media_player_manager.cc

[philipj_slow, 2014/03/26 17:58:04]

So this is a bit ugly, but it's still used in
BrowserMediaPlayerManager::OnProtectedSurfaceRequested(), it just has nothing to
do with the Blink setting any longer.

qinmin, what do you think should be done with that after this CL has landed? If
playback just doesn't work without fullscreen it seems to me like the
MediaPlayerMsg_RequestFullscreen should be unconditional, but I don't know the
details here.

If the flag should stay then a better name wouldn't hurt, but whichever product
sets the flag would have to be kept in sync, and that code is not available to
me.

 #endif
 
   prefs.touch_enabled = ui::AreTouchEventsEnabled();