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

Unified Diff: chrome/browser/profiles/profile_manager_browsertest.cc

Issue 654223009: Standardize usage of virtual/override/final in chrome/browser/profiles/ (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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/profiles/profile_manager_browsertest.cc
diff --git a/chrome/browser/profiles/profile_manager_browsertest.cc b/chrome/browser/profiles/profile_manager_browsertest.cc
index 059c37926e5ab6c37024f61c8ad71181a5267a33..c876244118c799adf18515d59f7243955d7aff55 100644
--- a/chrome/browser/profiles/profile_manager_browsertest.cc
+++ b/chrome/browser/profiles/profile_manager_browsertest.cc
@@ -69,7 +69,7 @@ class ProfileRemovalObserver : public ProfileInfoCacheObserver {
this);
}
- virtual ~ProfileRemovalObserver() {
+ ~ProfileRemovalObserver() override {
g_browser_process->profile_manager()->GetProfileInfoCache().RemoveObserver(
this);
}
@@ -77,8 +77,7 @@ class ProfileRemovalObserver : public ProfileInfoCacheObserver {
std::string last_used_profile_name() { return last_used_profile_name_; }
// ProfileInfoCacheObserver overrides:
- virtual void OnProfileWillBeRemoved(
- const base::FilePath& profile_path) override {
+ void OnProfileWillBeRemoved(const base::FilePath& profile_path) override {
last_used_profile_name_ = g_browser_process->local_state()->GetString(
prefs::kProfileLastUsed);
}
@@ -96,7 +95,7 @@ class PasswordStoreConsumerVerifier :
public:
PasswordStoreConsumerVerifier() : called_(false) {}
- virtual void OnGetPasswordStoreResults(
+ void OnGetPasswordStoreResults(
const std::vector<autofill::PasswordForm*>& results) override {
EXPECT_FALSE(called_);
called_ = true;
@@ -123,7 +122,7 @@ class PasswordStoreConsumerVerifier :
// platforms.
class ProfileManagerBrowserTest : public InProcessBrowserTest {
protected:
- virtual void SetUpCommandLine(CommandLine* command_line) override {
+ void SetUpCommandLine(CommandLine* command_line) override {
#if defined(OS_CHROMEOS)
command_line->AppendSwitch(
chromeos::switches::kIgnoreUserProfileMappingForTests);

Powered by Google App Engine
This is Rietveld 408576698