Remove the current /prefetch switch when relaunching Chrome.

chrome/common/switch_utils_unittest.cc

 // Copyright (c) 2011 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/switch_utils.h"
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/macros.h"
 #include "build/build_config.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 TEST(SwitchUtilsTest, RemoveSwitches) {
-  const base::CommandLine::CharType* argv[] = {
+  static const base::CommandLine::CharType* argv[] = {
       FILE_PATH_LITERAL("program"),
       FILE_PATH_LITERAL("--app=http://www.google.com/"),
       FILE_PATH_LITERAL("--force-first-run"),
       FILE_PATH_LITERAL("--make-default-browser"),
       FILE_PATH_LITERAL("--foo"),
       FILE_PATH_LITERAL("--bar")};
   base::CommandLine cmd_line(arraysize(argv), argv);
   EXPECT_FALSE(cmd_line.GetCommandLineString().empty());
 
   std::map<std::string, base::CommandLine::StringType> switches =
@@ skipping 21 matching lines @@
 
   std::map<std::string, base::CommandLine::StringType> switches =
       cmd_line.GetSwitches();
   EXPECT_EQ(5U, switches.size());
 
   switches::RemoveSwitchesForAutostart(&switches);
   EXPECT_EQ(2U, switches.size());
   EXPECT_TRUE(cmd_line.HasSwitch("foo"));
   EXPECT_TRUE(cmd_line.HasSwitch("bar"));
 }
+
+TEST(SwitchUtilsTest, RemovePrefetchSwitch) {
+  static const base::CommandLine::CharType* argv[] = {
+      FILE_PATH_LITERAL("program"),
+      FILE_PATH_LITERAL("--foo"),
+      FILE_PATH_LITERAL("/prefetch:1"),
+      FILE_PATH_LITERAL("--bar")};
+  base::CommandLine cmd_line(arraysize(argv), argv);
+  EXPECT_FALSE(cmd_line.GetCommandLineString().empty());
+
+  std::map<std::string, base::CommandLine::StringType> switches =
+      cmd_line.GetSwitches();
+  EXPECT_EQ(3U, switches.size());
+
+  switches::RemoveSwitchesForAutostart(&switches);
+  EXPECT_EQ(2U, switches.size());
+  EXPECT_TRUE(cmd_line.HasSwitch("foo"));
+  EXPECT_TRUE(cmd_line.HasSwitch("bar"));
+}
+
+TEST(SwitchUtilsTest, RemovePrefetchSwitches) {
+  static const base::CommandLine::CharType* argv[] = {
+      FILE_PATH_LITERAL("program"),
+      FILE_PATH_LITERAL("--foo"),
+      FILE_PATH_LITERAL("/prefetch:1"),
+      FILE_PATH_LITERAL("/prefetch:2"),

[gab, 2016/02/25 20:22:23]

As mentioned above, don't think we should support this scenario as it's an
invalid command-line (logic wise).

[fdoray, 2016/02/26 18:10:36]

Done.

+      FILE_PATH_LITERAL("--bar")};
+  base::CommandLine cmd_line(arraysize(argv), argv);
+  EXPECT_FALSE(cmd_line.GetCommandLineString().empty());
+
+  std::map<std::string, base::CommandLine::StringType> switches =
+      cmd_line.GetSwitches();
+  EXPECT_EQ(4U, switches.size());
+
+  switches::RemoveSwitchesForAutostart(&switches);
+  EXPECT_EQ(2U, switches.size());
+  EXPECT_TRUE(cmd_line.HasSwitch("foo"));
+  EXPECT_TRUE(cmd_line.HasSwitch("bar"));
+}
+
+TEST(SwitchUtilsTest, RemovePrefetchSwitchAndNormalSwitch) {
+  static const base::CommandLine::CharType* argv[] = {
+      FILE_PATH_LITERAL("program"),
+      FILE_PATH_LITERAL("--foo"),
+      FILE_PATH_LITERAL("/prefetch:1"),
+      FILE_PATH_LITERAL("--force-first-run"),
+      FILE_PATH_LITERAL("--bar")};
+  base::CommandLine cmd_line(arraysize(argv), argv);
+  EXPECT_FALSE(cmd_line.GetCommandLineString().empty());
+
+  std::map<std::string, base::CommandLine::StringType> switches =
+      cmd_line.GetSwitches();
+  EXPECT_EQ(4U, switches.size());
+
+  switches::RemoveSwitchesForAutostart(&switches);
+  EXPECT_EQ(2U, switches.size());
+  EXPECT_TRUE(cmd_line.HasSwitch("foo"));
+  EXPECT_TRUE(cmd_line.HasSwitch("bar"));
+}
 #endif