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

Side by Side Diff: chrome/browser/first_run/first_run_browsertest.cc

Issue 648653003: Standardize usage of virtual/override/final in chrome/browser/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months 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
« no previous file with comments | « chrome/browser/first_run/first_run.cc ('k') | chrome/browser/font_family_cache.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 <string> 5 #include <string>
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "base/prefs/pref_service.h" 10 #include "base/prefs/pref_service.h"
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 94
95 // This invokes BrowserMain, and does the import, so must be done last. 95 // This invokes BrowserMain, and does the import, so must be done last.
96 InProcessBrowserTest::SetUp(); 96 InProcessBrowserTest::SetUp();
97 } 97 }
98 98
99 virtual void TearDown() override { 99 virtual void TearDown() override {
100 EXPECT_TRUE(base::DeleteFile(prefs_file_, false)); 100 EXPECT_TRUE(base::DeleteFile(prefs_file_, false));
101 InProcessBrowserTest::TearDown(); 101 InProcessBrowserTest::TearDown();
102 } 102 }
103 103
104 virtual void SetUpCommandLine(CommandLine* command_line) override { 104 void SetUpCommandLine(CommandLine* command_line) override {
105 InProcessBrowserTest::SetUpCommandLine(command_line); 105 InProcessBrowserTest::SetUpCommandLine(command_line);
106 command_line->AppendSwitch(switches::kForceFirstRun); 106 command_line->AppendSwitch(switches::kForceFirstRun);
107 EXPECT_EQ(first_run::AUTO_IMPORT_NONE, first_run::auto_import_state()); 107 EXPECT_EQ(first_run::AUTO_IMPORT_NONE, first_run::auto_import_state());
108 108
109 extensions::ComponentLoader::EnableBackgroundExtensionsForTesting(); 109 extensions::ComponentLoader::EnableBackgroundExtensionsForTesting();
110 } 110 }
111 111
112 void SetMasterPreferencesForTest(const char text[]) { 112 void SetMasterPreferencesForTest(const char text[]) {
113 text_.reset(new std::string(text)); 113 text_.reset(new std::string(text));
114 } 114 }
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
242 " \"homepage\": \"example.com\",\n" 242 " \"homepage\": \"example.com\",\n"
243 " \"homepage_is_newtabpage\": false\n" 243 " \"homepage_is_newtabpage\": false\n"
244 "}\n"; 244 "}\n";
245 // A test fixture that will run in a first run scenario with master_preferences 245 // A test fixture that will run in a first run scenario with master_preferences
246 // set to kWithTrackedPrefs. Parameterizable on the SettingsEnforcement 246 // set to kWithTrackedPrefs. Parameterizable on the SettingsEnforcement
247 // experiment to be forced. 247 // experiment to be forced.
248 class FirstRunMasterPrefsWithTrackedPreferences 248 class FirstRunMasterPrefsWithTrackedPreferences
249 : public FirstRunMasterPrefsBrowserTestT<kWithTrackedPrefs>, 249 : public FirstRunMasterPrefsBrowserTestT<kWithTrackedPrefs>,
250 public testing::WithParamInterface<std::string> { 250 public testing::WithParamInterface<std::string> {
251 public: 251 public:
252 virtual void SetUpCommandLine(CommandLine* command_line) override { 252 void SetUpCommandLine(CommandLine* command_line) override {
253 FirstRunMasterPrefsBrowserTestT::SetUpCommandLine(command_line); 253 FirstRunMasterPrefsBrowserTestT::SetUpCommandLine(command_line);
254 command_line->AppendSwitchASCII( 254 command_line->AppendSwitchASCII(
255 switches::kForceFieldTrials, 255 switches::kForceFieldTrials,
256 std::string(chrome_prefs::internals::kSettingsEnforcementTrialName) + 256 std::string(chrome_prefs::internals::kSettingsEnforcementTrialName) +
257 "/" + GetParam() + "/"); 257 "/" + GetParam() + "/");
258 } 258 }
259 }; 259 };
260 260
261 // http://crbug.com/314221 261 // http://crbug.com/314221
262 #if defined(GOOGLE_CHROME_BUILD) && (defined(OS_MACOSX) || defined(OS_LINUX)) 262 #if defined(GOOGLE_CHROME_BUILD) && (defined(OS_MACOSX) || defined(OS_LINUX))
(...skipping 26 matching lines...) Expand all
289 FirstRunMasterPrefsWithTrackedPreferences, 289 FirstRunMasterPrefsWithTrackedPreferences,
290 testing::Values( 290 testing::Values(
291 chrome_prefs::internals::kSettingsEnforcementGroupNoEnforcement, 291 chrome_prefs::internals::kSettingsEnforcementGroupNoEnforcement,
292 chrome_prefs::internals::kSettingsEnforcementGroupEnforceAlways, 292 chrome_prefs::internals::kSettingsEnforcementGroupEnforceAlways,
293 chrome_prefs::internals:: 293 chrome_prefs::internals::
294 kSettingsEnforcementGroupEnforceAlwaysWithDSE, 294 kSettingsEnforcementGroupEnforceAlwaysWithDSE,
295 chrome_prefs::internals:: 295 chrome_prefs::internals::
296 kSettingsEnforcementGroupEnforceAlwaysWithExtensionsAndDSE)); 296 kSettingsEnforcementGroupEnforceAlwaysWithExtensionsAndDSE));
297 297
298 #endif // !defined(OS_CHROMEOS) 298 #endif // !defined(OS_CHROMEOS)
OLDNEW
« no previous file with comments | « chrome/browser/first_run/first_run.cc ('k') | chrome/browser/font_family_cache.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698