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

Unified Diff: ui/message_center/views/message_center_view.cc

Issue 14225010: Sets request_focus_on_press to false for message_center_buttons. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 7 years, 8 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 | « no previous file | ui/message_center/views/message_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..97af4224d15903b9ad28cdab3f72bfd00bbc4192 100644
--- a/ui/message_center/views/message_center_view.cc
+++ b/ui/message_center/views/message_center_view.cc
@@ -113,12 +113,15 @@ class NotificationCenterButton : public views::TextButton {
NotificationCenterButton(views::ButtonListener* listener,
const string16& text);
- protected:
// Overridden from views::View:
virtual gfx::Size GetPreferredSize() OVERRIDE;
- virtual void OnPaintBorder(gfx::Canvas* canvas);
- virtual void OnPaintFocusBorder(gfx::Canvas* canvas);
+ protected:
+ // Overridden from views::View:
+ virtual void OnPaintBorder(gfx::Canvas* canvas) OVERRIDE;
+ virtual void OnPaintFocusBorder(gfx::Canvas* canvas) OVERRIDE;
+
+ private:
DISALLOW_COPY_AND_ASSIGN(NotificationCenterButton);
};
@@ -172,7 +175,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 +199,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);
« no previous file with comments | « no previous file | ui/message_center/views/message_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698