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

Unified Diff: ui/message_center/views/notifier_settings_view.h

Issue 657923005: 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: ui/message_center/views/notifier_settings_view.h
diff --git a/ui/message_center/views/notifier_settings_view.h b/ui/message_center/views/notifier_settings_view.h
index ef60a06ab4427dad1db713cf5d407e144edaa6d2..394d60db8cc88fe0b24a124015a0ab35b71d27af 100644
--- a/ui/message_center/views/notifier_settings_view.h
+++ b/ui/message_center/views/notifier_settings_view.h
@@ -35,16 +35,16 @@ class MESSAGE_CENTER_EXPORT NotifierSettingsView
public views::MenuButtonListener {
public:
explicit NotifierSettingsView(NotifierSettingsProvider* provider);
- virtual ~NotifierSettingsView();
+ ~NotifierSettingsView() override;
bool IsScrollable();
// Overridden from NotifierSettingsDelegate:
- virtual void UpdateIconImage(const NotifierId& notifier_id,
- const gfx::Image& icon) override;
- virtual void NotifierGroupChanged() override;
- virtual void NotifierEnabledChanged(const NotifierId& notifier_id,
- bool enabled) override;
+ void UpdateIconImage(const NotifierId& notifier_id,
+ const gfx::Image& icon) override;
+ void NotifierGroupChanged() override;
+ void NotifierEnabledChanged(const NotifierId& notifier_id,
+ bool enabled) override;
void set_provider(NotifierSettingsProvider* new_provider) {
provider_ = new_provider;
@@ -59,7 +59,7 @@ class MESSAGE_CENTER_EXPORT NotifierSettingsView
NotifierButton(NotifierSettingsProvider* provider,
Notifier* notifier,
views::ButtonListener* listener);
- virtual ~NotifierButton();
+ ~NotifierButton() override;
void UpdateIconImage(const gfx::Image& icon);
void SetChecked(bool checked);
@@ -71,9 +71,8 @@ class MESSAGE_CENTER_EXPORT NotifierSettingsView
private:
// Overridden from views::ButtonListener:
- virtual void ButtonPressed(views::Button* button,
- const ui::Event& event) override;
- virtual void GetAccessibleState(ui::AXViewState* state) override;
+ void ButtonPressed(views::Button* button, const ui::Event& event) override;
+ void GetAccessibleState(ui::AXViewState* state) override;
bool ShouldHaveLearnMoreButton() const;
// Helper function to reset the layout when the view has substantially
@@ -96,19 +95,18 @@ class MESSAGE_CENTER_EXPORT NotifierSettingsView
void UpdateContentsView(const std::vector<Notifier*>& notifiers);
// Overridden from views::View:
- virtual void Layout() override;
- virtual gfx::Size GetMinimumSize() const override;
- virtual gfx::Size GetPreferredSize() const override;
- virtual bool OnKeyPressed(const ui::KeyEvent& event) override;
- virtual bool OnMouseWheel(const ui::MouseWheelEvent& event) override;
+ void Layout() override;
+ gfx::Size GetMinimumSize() const override;
+ gfx::Size GetPreferredSize() const override;
+ bool OnKeyPressed(const ui::KeyEvent& event) override;
+ bool OnMouseWheel(const ui::MouseWheelEvent& event) override;
// Overridden from views::ButtonListener:
- virtual void ButtonPressed(views::Button* sender,
- const ui::Event& event) override;
+ void ButtonPressed(views::Button* sender, const ui::Event& event) override;
// Overridden from views::MenuButtonListener:
- virtual void OnMenuButtonClicked(views::View* source,
- const gfx::Point& point) override;
+ void OnMenuButtonClicked(views::View* source,
+ const gfx::Point& point) override;
views::ImageButton* title_arrow_;
views::Label* title_label_;
« no previous file with comments | « ui/message_center/views/notification_view_unittest.cc ('k') | ui/message_center/views/notifier_settings_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698