Rename kDisableGestureRequirementForMediaPlayback and make it a test-only flag.

chromecast/browser/cast_browser_main_parts.cc

Index: chromecast/browser/cast_browser_main_parts.cc
diff --git a/chromecast/browser/cast_browser_main_parts.cc b/chromecast/browser/cast_browser_main_parts.cc
index ccb0fa874aa8c87c1a354ff3640c59041160c6de..29546c7b0718c75c0f0d8f25d1d861ccdc31fb11 100644
--- a/chromecast/browser/cast_browser_main_parts.cc
+++ b/chromecast/browser/cast_browser_main_parts.cc
@@ -216,48 +216,51 @@ struct DefaultCommandLineSwitch {
 
 DefaultCommandLineSwitch g_default_switches[] = {
 #if defined(OS_ANDROID)
-  // Disables Chromecast-specific WiFi-related features on ATV for now.
-  { switches::kNoWifi, "" },
-  { switches::kDisableGestureRequirementForMediaPlayback, ""},
-  { switches::kDisableMediaSuspend, ""},
+    // Disables Chromecast-specific WiFi-related features on ATV for now.
+    {switches::kNoWifi, ""},
+    // TODO(714676): this should probably set the no restrictions autoplay

[DaleCurtis, 2017/04/26 19:39:21]

Ditto on TODO formatting, also re-wrap comment.

[mlamouri (slow - plz ping), 2017/04/27 12:17:36]

Done. I will blame `git cl format` for the insane formatting :)

+    // policy
+    // instead.
+    {switches::kIgnoreAutoplayRestrictionsForTests, ""},
+    {switches::kDisableMediaSuspend, ""},
 #else
-  // GPU shader disk cache disabling is largely to conserve disk space.
-  { switches::kDisableGpuShaderDiskCache, "" },
-  // Enable media sessions by default (even on non-Android platforms).
-  { switches::kEnableDefaultMediaSession, "" },
+    // GPU shader disk cache disabling is largely to conserve disk space.
+    {switches::kDisableGpuShaderDiskCache, ""},
+    // Enable media sessions by default (even on non-Android platforms).
+    {switches::kEnableDefaultMediaSession, ""},
 #endif
 #if BUILDFLAG(IS_CAST_AUDIO_ONLY)
 #if defined(OS_ANDROID)
-  { switches::kDisableGLDrawingForTests, "" },
+    {switches::kDisableGLDrawingForTests, ""},
 #else
-  { switches::kDisableGpu, "" },
+    {switches::kDisableGpu, ""},
 #endif  // defined(OS_ANDROID)
 #endif  // BUILDFLAG(IS_CAST_AUDIO_ONLY)
 #if defined(OS_LINUX)
 #if defined(ARCH_CPU_X86_FAMILY)
-  // This is needed for now to enable the x11 Ozone platform to work with
-  // current Linux/NVidia OpenGL drivers.
-  { switches::kIgnoreGpuBlacklist, ""},
+    // This is needed for now to enable the x11 Ozone platform to work with
+    // current Linux/NVidia OpenGL drivers.
+    {switches::kIgnoreGpuBlacklist, ""},
 #elif defined(ARCH_CPU_ARM_FAMILY)
 #if !BUILDFLAG(IS_CAST_AUDIO_ONLY)
-  {switches::kEnableHardwareOverlays, "cast"},
+    {switches::kEnableHardwareOverlays, "cast"},
 #endif
 #endif
 #endif  // defined(OS_LINUX)
-  // Needed so that our call to GpuDataManager::SetGLStrings doesn't race
-  // against GPU process creation (which is otherwise triggered from
-  // BrowserThreadsStarted).  The GPU process will be created as soon as a
-  // renderer needs it, which always happens after main loop starts.
-  { switches::kDisableGpuEarlyInit, "" },
-  // TODO(halliwell): Cast builds don't support ES3. Remove this switch when
-  // support is added (crbug.com/659395)
-  { switches::kDisableES3GLContext, "" },
-  // Enable navigator.connection API.
-  // TODO(derekjchow): Remove this switch when enabled by default.
-  { switches::kEnableNetworkInformation, "" },
-  // TODO(halliwell): Remove after fixing b/35422666.
-  { switches::kEnableUseZoomForDSF, "false" },
-  { NULL, NULL },  // Termination
+    // Needed so that our call to GpuDataManager::SetGLStrings doesn't race
+    // against GPU process creation (which is otherwise triggered from
+    // BrowserThreadsStarted).  The GPU process will be created as soon as a
+    // renderer needs it, which always happens after main loop starts.
+    {switches::kDisableGpuEarlyInit, ""},
+    // TODO(halliwell): Cast builds don't support ES3. Remove this switch when
+    // support is added (crbug.com/659395)
+    {switches::kDisableES3GLContext, ""},
+    // Enable navigator.connection API.
+    // TODO(derekjchow): Remove this switch when enabled by default.
+    {switches::kEnableNetworkInformation, ""},
+    // TODO(halliwell): Remove after fixing b/35422666.
+    {switches::kEnableUseZoomForDSF, "false"},
+    {NULL, NULL},  // Termination

[DaleCurtis, 2017/04/26 19:39:21]

Might as well make it nullptr, nullptr while you're here.

[mlamouri (slow - plz ping), 2017/04/27 12:17:36]

Done. I will again blame `git cl format` for re-formatting all of this. Will do
the nullptr change :)

 };
 
 void AddDefaultCommandLineSwitches(base::CommandLine* command_line) {