Chromium Code Reviews| Index: ui/message_center/views/message_center_view.cc |
| diff --git a/ui/message_center/views/message_center_view.cc b/ui/message_center/views/message_center_view.cc |
| index c47c9a7327330b92ca7dff09df6cef8f348fd75f..c505d23fda17ce9cabbf3e042b764ec6838699df 100644 |
| --- a/ui/message_center/views/message_center_view.cc |
| +++ b/ui/message_center/views/message_center_view.cc |
| @@ -113,12 +113,12 @@ class NotificationCenterButton : public views::TextButton { |
| NotificationCenterButton(views::ButtonListener* listener, |
| const string16& text); |
| - protected: |
|
dharcourt
2013/04/17 21:58:56
"protected:" should be moved down two lines instea
Jun Mukai
2013/04/18 18:16:44
Done.
|
| // Overridden from views::View: |
| virtual gfx::Size GetPreferredSize() OVERRIDE; |
| virtual void OnPaintBorder(gfx::Canvas* canvas); |
| virtual void OnPaintFocusBorder(gfx::Canvas* canvas); |
|
dharcourt
2013/04/17 21:58:56
I know this is preexisting, but shouldn't OnPaintB
Jun Mukai
2013/04/18 18:16:44
Done.
|
| + private: |
| DISALLOW_COPY_AND_ASSIGN(NotificationCenterButton); |
| }; |
| @@ -172,7 +172,7 @@ class RichMessageCenterButtonBar : public MessageCenterButtonBar, |
| const ui::Event& event) OVERRIDE; |
| views::Label* notification_label_; |
| - views::Button* settings_button_; |
| + NotificationCenterButton* settings_button_; |
| DISALLOW_COPY_AND_ASSIGN(RichMessageCenterButtonBar); |
| }; |
| @@ -196,10 +196,12 @@ RichMessageCenterButtonBar::RichMessageCenterButtonBar( |
| this, l10n_util::GetStringUTF16( |
| IDS_MESSAGE_CENTER_SETTINGS_BUTTON_LABEL)); |
| settings_button_->set_focusable(true); |
| + settings_button_->set_request_focus_on_press(false); |
| AddChildView(settings_button_); |
| NotificationCenterButton* close_all_button = new NotificationCenterButton( |
| this, l10n_util::GetStringUTF16(IDS_MESSAGE_CENTER_CLEAR_ALL)); |
| close_all_button->set_focusable(true); |
| + close_all_button->set_request_focus_on_press(false); |
| AddChildView(close_all_button); |
| views::GridLayout* layout = new views::GridLayout(this); |