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

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

Issue 684613002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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 (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 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 76
77 // A generic test class to be subclassed by test classes testing specific 77 // A generic test class to be subclassed by test classes testing specific
78 // master_preferences. All subclasses must call SetMasterPreferencesForTest() 78 // master_preferences. All subclasses must call SetMasterPreferencesForTest()
79 // from their SetUp() method before deferring the remainder of Setup() to this 79 // from their SetUp() method before deferring the remainder of Setup() to this
80 // class. 80 // class.
81 class FirstRunMasterPrefsBrowserTestBase : public InProcessBrowserTest { 81 class FirstRunMasterPrefsBrowserTestBase : public InProcessBrowserTest {
82 public: 82 public:
83 FirstRunMasterPrefsBrowserTestBase() {} 83 FirstRunMasterPrefsBrowserTestBase() {}
84 84
85 protected: 85 protected:
86 virtual void SetUp() override { 86 void SetUp() override {
87 // All users of this test class need to call SetMasterPreferencesForTest() 87 // All users of this test class need to call SetMasterPreferencesForTest()
88 // before this class' SetUp() is invoked. 88 // before this class' SetUp() is invoked.
89 ASSERT_TRUE(text_.get()); 89 ASSERT_TRUE(text_.get());
90 90
91 ASSERT_TRUE(base::CreateTemporaryFile(&prefs_file_)); 91 ASSERT_TRUE(base::CreateTemporaryFile(&prefs_file_));
92 EXPECT_TRUE(base::WriteFile(prefs_file_, text_->c_str(), text_->size())); 92 EXPECT_TRUE(base::WriteFile(prefs_file_, text_->c_str(), text_->size()));
93 first_run::SetMasterPrefsPathForTesting(prefs_file_); 93 first_run::SetMasterPrefsPathForTesting(prefs_file_);
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 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 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();
(...skipping 179 matching lines...) Expand 10 before | Expand all | Expand 10 after
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

Powered by Google App Engine
This is Rietveld 408576698