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

Unified Diff: chrome/browser/sync/sync_error_notifier_ash_unittest.cc

Issue 686563003: Standardize usage of virtual/override/final specifiers. (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/sync/sync_error_notifier_ash_unittest.cc
diff --git a/chrome/browser/sync/sync_error_notifier_ash_unittest.cc b/chrome/browser/sync/sync_error_notifier_ash_unittest.cc
index 1b4f5e712cb008964e09b08eedf65bae3a1cb263..a4ad85997b138bf3ab1aca4f70f8bb907ae49bc6 100644
--- a/chrome/browser/sync/sync_error_notifier_ash_unittest.cc
+++ b/chrome/browser/sync/sync_error_notifier_ash_unittest.cc
@@ -65,23 +65,21 @@ class ScreenTypeDelegateDesktop : public gfx::ScreenTypeDelegate {
class FakeLoginUIService: public LoginUIService {
public:
FakeLoginUIService() : LoginUIService(NULL) {}
- virtual ~FakeLoginUIService() {}
+ ~FakeLoginUIService() override {}
};
class FakeLoginUI : public LoginUIService::LoginUI {
public:
FakeLoginUI() : focus_ui_call_count_(0) {}
- virtual ~FakeLoginUI() {}
+ ~FakeLoginUI() override {}
int focus_ui_call_count() const { return focus_ui_call_count_; }
private:
// LoginUIService::LoginUI:
- virtual void FocusUI() override {
- ++focus_ui_call_count_;
- }
- virtual void CloseUI() override {}
+ void FocusUI() override { ++focus_ui_call_count_; }
+ void CloseUI() override {}
int focus_ui_call_count_;
};
@@ -94,9 +92,9 @@ KeyedService* BuildMockLoginUIService(
class SyncErrorNotifierTest : public AshTestBase {
public:
SyncErrorNotifierTest() {}
- virtual ~SyncErrorNotifierTest() {}
+ ~SyncErrorNotifierTest() override {}
- virtual void SetUp() override {
+ void SetUp() override {
profile_manager_.reset(
new TestingProfileManager(TestingBrowserProcess::GetGlobal()));
ASSERT_TRUE(profile_manager_->SetUp());
@@ -128,7 +126,7 @@ class SyncErrorNotifierTest : public AshTestBase {
notification_ui_manager_ = g_browser_process->notification_ui_manager();
}
- virtual void TearDown() override {
+ void TearDown() override {
error_notifier_->Shutdown();
service_.reset();
#if defined(OS_WIN)
« no previous file with comments | « chrome/browser/sync/sync_error_notifier_ash.cc ('k') | chrome/browser/sync/sync_error_notifier_factory_ash.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698