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

Unified Diff: chrome/browser/ui/views/toolbar/wrench_toolbar_button.cc

Issue 298813002: views: Move MenuButton from TextButton to LabelButton. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Disable DragDirectlyToSecondWindow. Created 6 years, 6 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/ui/views/toolbar/wrench_toolbar_button.cc
diff --git a/chrome/browser/ui/views/toolbar/wrench_toolbar_button.cc b/chrome/browser/ui/views/toolbar/wrench_toolbar_button.cc
index c3c5d7dbadcfd1f634906ba0fbcbb8eb26add41c..c096597ab39e5e2a1c79f662d789f5644aceae10 100644
--- a/chrome/browser/ui/views/toolbar/wrench_toolbar_button.cc
+++ b/chrome/browser/ui/views/toolbar/wrench_toolbar_button.cc
@@ -7,14 +7,11 @@
#include "grit/theme_resources.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/base/theme_provider.h"
+#include "ui/views/painter.h"
WrenchToolbarButton::WrenchToolbarButton(views::MenuButtonListener* listener)
: views::MenuButton(NULL, base::string16(), listener, false) {
wrench_icon_painter_.reset(new WrenchIconPainter(this));
-
- // Used for sizing only.
- ui::ResourceBundle& rb = ResourceBundle::GetSharedInstance();
- SetIcon(*rb.GetImageSkiaNamed(IDR_TOOLBAR_BEZEL_HOVER));
}
WrenchToolbarButton::~WrenchToolbarButton() {
@@ -25,6 +22,11 @@ void WrenchToolbarButton::SetSeverity(WrenchIconPainter::Severity severity,
wrench_icon_painter_->SetSeverity(severity, animate);
}
+gfx::Size WrenchToolbarButton::GetPreferredSize() const {
+ return ResourceBundle::GetSharedInstance().
+ GetImageSkiaNamed(IDR_TOOLBAR_BEZEL_HOVER)->size();
+}
+
void WrenchToolbarButton::OnPaint(gfx::Canvas* canvas) {
wrench_icon_painter_->Paint(
canvas, GetThemeProvider(), gfx::Rect(size()), GetCurrentBezelType());

Powered by Google App Engine
This is Rietveld 408576698