Android: Remove the prompt when entering fullscreen mode (behind flag).

chrome/common/chrome_features.cc

 // Copyright 2016 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.
 
 #include "chrome/common/chrome_features.h"
 
 namespace features {
 
 // All features in alphabetical order.
 
 #if defined(OS_WIN) || defined(OS_MACOSX)
 // Enables automatic tab discarding, when the system is in low memory state.
 const base::Feature kAutomaticTabDiscarding{"AutomaticTabDiscarding",
                                             base::FEATURE_DISABLED_BY_DEFAULT};
 #endif  // defined(OS_WIN) || defined(OS_MACOSX)
 
 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_LINUX) && !defined(OS_CHROMEOS)
 // Enables showing the "This computer will no longer receive Google Chrome
 // updates" infobar instead of the "will soon stop receiving" infobar on
 // deprecated systems.
 const base::Feature kLinuxObsoleteSystemIsEndOfTheLine{
     "LinuxObsoleteSystemIsEndOfTheLine", base::FEATURE_DISABLED_BY_DEFAULT};
 #endif
 
+// A new user experience for transitioning into fullscreen and mouse pointer
+// lock states. The name is a misnomer (for historical reasons); affects both
+// Views and Android builds.
+const base::Feature kSimplifiedFullscreenUI{

[jochen (gone - plz use gerrit), 2016/03/04 12:54:39]

is this clang-formatted?

[Matt Giuca, 2016/03/04 23:00:28]

Not sure (I was following the pattern for the rest of the file). I'll check when
back at the office.

[Matt Giuca, 2016/03/07 05:53:09]

Uh yeah, it is "kind of" clang formatted.

Without the #if defineds it formats the same as the above ones (brace without a
space; I guess intended to look like a function call). If I run clang-format
over this I get:


const base::Feature kSimplifiedFullscreenUI {
  "ViewsSimplifiedFullscreenUI",
#if defined(USE_AURA)
      // Windows, Linux, Chrome OS.
      base::FEATURE_ENABLED_BY_DEFAULT,
#else
      // Mac, Android.
      base::FEATURE_DISABLED_BY_DEFAULT,
#endif

which just looks wrong. So keeping it as-is.

+    "ViewsSimplifiedFullscreenUI",
+#if defined(USE_AURA)
+    // Windows, Linux, Chrome OS.
+    base::FEATURE_ENABLED_BY_DEFAULT,
+#else
+    // Mac, Android.
+    base::FEATURE_DISABLED_BY_DEFAULT,
+#endif
+};
+
 }  // namespace features