Prefix CommandLine usage with base namespace (Part 9: components)

components/google/core/browser/google_util_unittest.cc

 // Copyright (c) 2012 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 "base/command_line.h"
 #include "components/google/core/browser/google_switches.h"
 #include "components/google/core/browser/google_url_tracker.h"
 #include "components/google/core/browser/google_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 292 matching lines @@
   EXPECT_FALSE(IsGoogleDomainUrl(GURL("http://www.foo.com/xyz"),
                                  google_util::DISALLOW_SUBDOMAIN,
                                  google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_FALSE(IsGoogleDomainUrl(GURL("https://www.foo.com/"),
                                  google_util::DISALLOW_SUBDOMAIN,
                                  google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_FALSE(IsHomePage("https://www.foo.com/webhp"));
   EXPECT_FALSE(IsSearch("http://www.foo.com/search?q=a"));
 
   // Override the Google base URL on the command line.
-  CommandLine::ForCurrentProcess()->AppendSwitchASCII(switches::kGoogleBaseURL,
-                                                      "http://www.foo.com/");
+  base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
+                            switches::kGoogleBaseURL, "http://www.foo.com/");

[Cait (Slow), 2014/12/03 16:43:21]

indent is incorrect here (and below). Should be 4 spaces.

 
   // Only URLs which start with exactly the string on the command line should
   // cause StartsWithCommandLineGoogleBaseURL() to return true.
   EXPECT_FALSE(StartsWithBaseURL(std::string()));
   EXPECT_TRUE(StartsWithBaseURL("http://www.foo.com/"));
   EXPECT_TRUE(StartsWithBaseURL("http://www.foo.com/abc"));
   EXPECT_FALSE(StartsWithBaseURL("https://www.foo.com/"));
   EXPECT_FALSE(StartsWithBaseURL("http://www.google.com/"));
 
   // The various IsGoogleXXX functions should respect the command-line flag.
   EXPECT_TRUE(IsGoogleHostname("www.foo.com", google_util::DISALLOW_SUBDOMAIN));
   EXPECT_FALSE(IsGoogleHostname("foo.com", google_util::ALLOW_SUBDOMAIN));
   EXPECT_TRUE(IsGoogleDomainUrl(GURL("http://www.foo.com/xyz"),
                                 google_util::DISALLOW_SUBDOMAIN,
                                 google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_TRUE(IsGoogleDomainUrl(GURL("https://www.foo.com/"),
                                 google_util::DISALLOW_SUBDOMAIN,
                                 google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_TRUE(IsHomePage("https://www.foo.com/webhp"));
   EXPECT_FALSE(IsHomePage("http://www.foo.com/xyz"));
   EXPECT_TRUE(IsSearch("http://www.foo.com/search?q=a"));
 }
 
 TEST(GoogleUtilTest, GoogleBaseURLDisallowQuery) {
-  CommandLine::ForCurrentProcess()->AppendSwitchASCII(switches::kGoogleBaseURL,
-                                                      "http://www.foo.com/?q=");
+  base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
+                            switches::kGoogleBaseURL, "http://www.foo.com/?q=");
   EXPECT_FALSE(google_util::CommandLineGoogleBaseURL().is_valid());
 }
 
 TEST(GoogleUtilTest, GoogleBaseURLDisallowRef) {
-  CommandLine::ForCurrentProcess()->AppendSwitchASCII(switches::kGoogleBaseURL,
-                                                      "http://www.foo.com/#q=");
+  base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
+                            switches::kGoogleBaseURL, "http://www.foo.com/#q=");
   EXPECT_FALSE(google_util::CommandLineGoogleBaseURL().is_valid());
 }
 
 TEST(GoogleUtilTest, GoogleBaseURLFixup) {
-  CommandLine::ForCurrentProcess()->AppendSwitchASCII(switches::kGoogleBaseURL,
-                                                      "www.foo.com");
+  base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
+                            switches::kGoogleBaseURL, "www.foo.com");
   ASSERT_TRUE(google_util::CommandLineGoogleBaseURL().is_valid());
   EXPECT_EQ("http://www.foo.com/",
             google_util::CommandLineGoogleBaseURL().spec());
 }
 
 TEST(GoogleUtilTest, YoutubeDomains) {
   EXPECT_TRUE(IsYoutubeDomainUrl(GURL("http://www.youtube.com"),
                                  google_util::ALLOW_SUBDOMAIN,
                                  google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_TRUE(IsYoutubeDomainUrl(GURL("http://youtube.com"),
@@ skipping 35 matching lines @@
   EXPECT_FALSE(IsYoutubeDomainUrl(GURL("http://www.youtube.com:123"),
                                   google_util::DISALLOW_SUBDOMAIN,
                                   google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_TRUE(IsYoutubeDomainUrl(GURL("http://www.youtube.com:123"),
                                  google_util::DISALLOW_SUBDOMAIN,
                                  google_util::ALLOW_NON_STANDARD_PORTS));
   EXPECT_FALSE(IsYoutubeDomainUrl(GURL("file://www.youtube.com"),
                                   google_util::DISALLOW_SUBDOMAIN,
                                   google_util::DISALLOW_NON_STANDARD_PORTS));
 }