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

Unified Diff: chrome/browser/views/wrench_menu.cc

Issue 3058011: Add "pushed" as a state a TextButton can show (alongside "normal" and "hover"... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 5 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/views/wrench_menu.cc
===================================================================
--- chrome/browser/views/wrench_menu.cc (revision 53742)
+++ chrome/browser/views/wrench_menu.cc (working copy)
@@ -60,7 +60,8 @@
// Subclass of ImageButton whose preferred size includes the size of the border.
class FullscreenButton : public ImageButton {
public:
- FullscreenButton(views::ButtonListener* listener) : ImageButton(listener) {}
+ explicit FullscreenButton(views::ButtonListener* listener)
+ : ImageButton(listener) { }
virtual gfx::Size GetPreferredSize() {
gfx::Size pref = ImageButton::GetPreferredSize();
@@ -258,7 +259,6 @@
*background = bg;
button->set_border(new MenuButtonBorder());
button->set_alignment(TextButton::ALIGN_CENTER);
- button->SetShowHighlighted(true);
button->SetNormalHasBorder(true);
button->SetFont(views::MenuConfig::instance().font);
button->ClearMaxTextSize();

Powered by Google App Engine
This is Rietveld 408576698