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

Unified Diff: ash/system/web_notification/web_notification_tray_unittest.cc

Issue 683473005: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase 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
« no previous file with comments | « ash/system/web_notification/web_notification_tray.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/system/web_notification/web_notification_tray_unittest.cc
diff --git a/ash/system/web_notification/web_notification_tray_unittest.cc b/ash/system/web_notification/web_notification_tray_unittest.cc
index 5fa566129f25667ed643d69094fd8880934f6916..d5bad51740820089987e388dbdbfb10aacb82ec4 100644
--- a/ash/system/web_notification/web_notification_tray_unittest.cc
+++ b/ash/system/web_notification/web_notification_tray_unittest.cc
@@ -72,15 +72,14 @@ class TestItem : public SystemTrayItem {
public:
TestItem() : SystemTrayItem(GetSystemTray()) {}
- virtual views::View* CreateDefaultView(user::LoginStatus status) override {
+ views::View* CreateDefaultView(user::LoginStatus status) override {
views::View* default_view = new views::View;
default_view->SetLayoutManager(new views::FillLayout);
default_view->AddChildView(new views::Label(base::UTF8ToUTF16("Default")));
return default_view;
}
- virtual views::View* CreateNotificationView(
- user::LoginStatus status) override {
+ views::View* CreateNotificationView(user::LoginStatus status) override {
return new views::View;
}
@@ -93,15 +92,15 @@ class TestItem : public SystemTrayItem {
class WebNotificationTrayTest : public test::AshTestBase {
public:
WebNotificationTrayTest() {}
- virtual ~WebNotificationTrayTest() {}
+ ~WebNotificationTrayTest() override {}
- virtual void SetUp() override {
+ void SetUp() override {
CommandLine::ForCurrentProcess()->AppendSwitch(
switches::kEnableTouchFeedback);
test::AshTestBase::SetUp();
}
- virtual void TearDown() override {
+ void TearDown() override {
GetMessageCenter()->RemoveAllNotifications(false);
test::AshTestBase::TearDown();
}
« no previous file with comments | « ash/system/web_notification/web_notification_tray.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698