Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(232)

Side by Side Diff: components/search_engines/template_url_prepopulate_data_unittest.cc

Issue 818103002: Make callers of CommandLine use it via the base:: namespace. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/command_line.h" 5 #include "base/command_line.h"
6 #include "base/files/scoped_temp_dir.h" 6 #include "base/files/scoped_temp_dir.h"
7 #include "base/memory/scoped_vector.h" 7 #include "base/memory/scoped_vector.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "components/google/core/browser/google_switches.h" 9 #include "components/google/core/browser/google_switches.h"
10 #include "components/pref_registry/testing_pref_service_syncable.h" 10 #include "components/pref_registry/testing_pref_service_syncable.h"
(...skipping 325 matching lines...) Expand 10 before | Expand all | Expand 10 after
336 336
337 // Search URL for which no prepopulated search provider exists. 337 // Search URL for which no prepopulated search provider exists.
338 EXPECT_EQ(SEARCH_ENGINE_OTHER, 338 EXPECT_EQ(SEARCH_ENGINE_OTHER,
339 GetEngineType("http://example.net/search?q={searchTerms}")); 339 GetEngineType("http://example.net/search?q={searchTerms}"));
340 EXPECT_EQ(SEARCH_ENGINE_OTHER, GetEngineType("invalid:search:url")); 340 EXPECT_EQ(SEARCH_ENGINE_OTHER, GetEngineType("invalid:search:url"));
341 341
342 // URL that doesn't look Google-related, but matches a Google base URL 342 // URL that doesn't look Google-related, but matches a Google base URL
343 // specified on the command line. 343 // specified on the command line.
344 const std::string foo_url("http://www.foo.com/search?q={searchTerms}"); 344 const std::string foo_url("http://www.foo.com/search?q={searchTerms}");
345 EXPECT_EQ(SEARCH_ENGINE_OTHER, GetEngineType(foo_url)); 345 EXPECT_EQ(SEARCH_ENGINE_OTHER, GetEngineType(foo_url));
346 CommandLine::ForCurrentProcess()->AppendSwitchASCII(switches::kGoogleBaseURL, 346 base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
347 "http://www.foo.com/"); 347 switches::kGoogleBaseURL, "http://www.foo.com/");
348 EXPECT_EQ(SEARCH_ENGINE_GOOGLE, GetEngineType(foo_url)); 348 EXPECT_EQ(SEARCH_ENGINE_GOOGLE, GetEngineType(foo_url));
349 } 349 }
OLDNEW
« no previous file with comments | « components/search_engines/template_url.cc ('k') | components/search_engines/template_url_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698