Completely disable about:flags

This is an attempt to improve stability on the m14 branch. I will revert this CL right after merging it to m14.

BUG=95592
TEST=Go to about:flags and chrome://flags. Nothing should be there.
Go to about:version, no --begin-flags and --end-flags should be there.

Review URL: http://codereview.chromium.org/7837006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99883 0039d316-1c4b-4281-b951-d872f2087c98

5 files changed, 36 insertions, 15 deletions

diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index 93d9358..286c72a 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -697,7 +697,10 @@ void FlagsState::ConvertFlagsToSwitches(
     }
   }
 
+// TODO(thakis): Reenable.
+#if 0
   command_line->AppendSwitch(switches::kFlagSwitchesBegin);
+#endif
   flags_switches_.insert(
       std::pair<std::string, std::string>(switches::kFlagSwitchesBegin,
                                           std::string()));
@@ -715,11 +718,17 @@ void FlagsState::ConvertFlagsToSwitches(
     const std::pair<std::string, std::string>&
         switch_and_value_pair = name_to_switch_it->second;
 
+// TODO(thakis): Reenable.
+#if 0
     command_line->AppendSwitchASCII(switch_and_value_pair.first,
                                     switch_and_value_pair.second);
+#endif
     flags_switches_[switch_and_value_pair.first] = switch_and_value_pair.second;
   }
+// TODO(thakis): Reenable.
+#if 0
   command_line->AppendSwitch(switches::kFlagSwitchesEnd);
+#endif
   flags_switches_.insert(
       std::pair<std::string, std::string>(switches::kFlagSwitchesEnd,
                                           std::string()));
diff --git a/chrome/browser/about_flags_unittest.cc b/chrome/browser/about_flags_unittest.cc
index bf54c1f..6c4dd6f 100644
--- a/chrome/browser/about_flags_unittest.cc
+++ b/chrome/browser/about_flags_unittest.cc
@@ -183,7 +183,8 @@ TEST_F(AboutFlagsTest, ConvertFlagsToSwitches) {
   ConvertFlagsToSwitches(&prefs_, &command_line);
 
   EXPECT_TRUE(command_line.HasSwitch("foo"));
-  EXPECT_TRUE(command_line.HasSwitch(kSwitch1));
+  // TODO(thakis): Undo
+  EXPECT_FALSE(command_line.HasSwitch(kSwitch1));
 }
 
 TEST_F(AboutFlagsTest, RemoveFlagSwitches) {
@@ -228,7 +229,8 @@ TEST_F(AboutFlagsTest, PersistAndPrune) {
   // Convert the flags to switches. Experiment 3 shouldn't be among the switches
   // as it is not applicable to the current platform.
   ConvertFlagsToSwitches(&prefs_, &command_line);
-  EXPECT_TRUE(command_line.HasSwitch(kSwitch1));
+  // TODO(thakis): undo.
+  EXPECT_FALSE(command_line.HasSwitch(kSwitch1));
   EXPECT_FALSE(command_line.HasSwitch(kSwitch3));
 
   // Experiment 3 should show still be persisted in preferences though.
@@ -244,15 +246,18 @@ TEST_F(AboutFlagsTest, CheckValues) {
   SetExperimentEnabled(&prefs_, kFlags1, true);
   SetExperimentEnabled(&prefs_, kFlags2, true);
   CommandLine command_line(CommandLine::NO_PROGRAM);
+  // TODO(thakis): undo.
   EXPECT_FALSE(command_line.HasSwitch(kSwitch1));
   EXPECT_FALSE(command_line.HasSwitch(kSwitch2));
 
   // Convert the flags to switches.
   ConvertFlagsToSwitches(&prefs_, &command_line);
-  EXPECT_TRUE(command_line.HasSwitch(kSwitch1));
+  EXPECT_FALSE(command_line.HasSwitch(kSwitch1));
   EXPECT_EQ(std::string(""), command_line.GetSwitchValueASCII(kSwitch1));
-  EXPECT_TRUE(command_line.HasSwitch(kSwitch2));
-  EXPECT_EQ(std::string(kValueForSwitch2),
+  // TODO(thakis): undo.
+  EXPECT_FALSE(command_line.HasSwitch(kSwitch2));
+  // TODO(thakis): undo.
+  EXPECT_EQ("",
             command_line.GetSwitchValueASCII(kSwitch2));
 
   // Confirm that there is no '=' in the command line for simple switches.
@@ -273,11 +278,13 @@ TEST_F(AboutFlagsTest, CheckValues) {
                                     std::string(kSwitch2) +
                                     std::string("=");
 #if defined(OS_WIN)            
-  EXPECT_NE(std::wstring::npos,
+  // TODO(thakis): undo.
+  EXPECT_EQ(std::wstring::npos,
             command_line.GetCommandLineString().find(
                 ASCIIToWide(switch2_with_equals)));
 #else
-  EXPECT_NE(std::string::npos,
+  // TODO(thakis): undo.
+  EXPECT_EQ(std::string::npos,
             command_line.GetCommandLineString().find(switch2_with_equals));
 #endif
 
@@ -306,8 +313,10 @@ TEST_F(AboutFlagsTest, MultiValues) {
     CommandLine command_line(CommandLine::NO_PROGRAM);
     ConvertFlagsToSwitches(&prefs_, &command_line);
     EXPECT_FALSE(command_line.HasSwitch(kMultiSwitch1));
-    EXPECT_TRUE(command_line.HasSwitch(kMultiSwitch2));
-    EXPECT_EQ(std::string(kValueForMultiSwitch2),
+    // TODO(thakis): undo.
+    EXPECT_FALSE(command_line.HasSwitch(kMultiSwitch2));
+    // TODO(thakis): undo.
+    EXPECT_EQ("",
               command_line.GetSwitchValueASCII(kMultiSwitch2));
   }
 
diff --git a/chrome/browser/autocomplete/builtin_provider_unittest.cc b/chrome/browser/autocomplete/builtin_provider_unittest.cc
index 8847bf4..463388d4 100644
--- a/chrome/browser/autocomplete/builtin_provider_unittest.cc
+++ b/chrome/browser/autocomplete/builtin_provider_unittest.cc
@@ -131,7 +131,7 @@ TEST_F(BuiltinProviderTest, NonChromeURLs) {
                 &AutocompleteMatch::destination_url);
 }
 
-TEST_F(BuiltinProviderTest, ChromeURLs) {
+TEST_F(BuiltinProviderTest, DISABLED_ChromeURLs) {
   const string16 kAbout = ASCIIToUTF16(chrome::kAboutScheme);
   const string16 kChrome = ASCIIToUTF16(chrome::kChromeUIScheme);
   const string16 kSeparator1 = ASCIIToUTF16(":");
diff --git a/chrome/browser/browser_about_handler.cc b/chrome/browser/browser_about_handler.cc
index 12661de..5e4d476 100644
--- a/chrome/browser/browser_about_handler.cc
+++ b/chrome/browser/browser_about_handler.cc
@@ -120,7 +120,8 @@ const char* const kChromePaths[] = {
   chrome::kChromeUIDNSHost,
   chrome::kChromeUIDownloadsHost,
   chrome::kChromeUIExtensionsHost,
-  chrome::kChromeUIFlagsHost,
+  // TODO(thakis): Reenable.
+  //chrome::kChromeUIFlagsHost,
   chrome::kChromeUIFlashHost,
   chrome::kChromeUIGpuInternalsHost,
   chrome::kChromeUIHistogramsHost,
diff --git a/chrome/browser/ui/webui/chrome_web_ui_factory.cc b/chrome/browser/ui/webui/chrome_web_ui_factory.cc
index 37fbec2..500533c 100644
--- a/chrome/browser/ui/webui/chrome_web_ui_factory.cc
+++ b/chrome/browser/ui/webui/chrome_web_ui_factory.cc
@@ -180,8 +180,9 @@ static WebUIFactoryFunction GetWebUIFactoryFunction(Profile* profile,
     return &NewWebUI<HistoryUI>;
   if (url.host() == chrome::kChromeUIHistory2Host)
     return &NewWebUI<HistoryUI2>;
-  if (url.host() == chrome::kChromeUIFlagsHost)
-    return &NewWebUI<FlagsUI>;
+  // TODO(thakis): Reenable.
+  //if (url.host() == chrome::kChromeUIFlagsHost)
+  //  return &NewWebUI<FlagsUI>;
   if (url.host() == chrome::kChromeUIFlashHost)
     return &NewWebUI<FlashUI>;
 #if defined(TOUCH_UI)
@@ -387,8 +388,9 @@ RefCountedMemory* ChromeWebUIFactory::GetFaviconResourceBytes(
   if (page_url.host() == chrome::kChromeUIHistory2Host)
     return HistoryUI2::GetFaviconResourceBytes();
 
-  if (page_url.host() == chrome::kChromeUIFlagsHost)
-    return FlagsUI::GetFaviconResourceBytes();
+  // TODO(thakis): Reenable.
+  //if (page_url.host() == chrome::kChromeUIFlagsHost)
+  //  return FlagsUI::GetFaviconResourceBytes();
 
   if (page_url.host() == chrome::kChromeUISessionsHost)
     return SessionsUI::GetFaviconResourceBytes();