diff options
| author | cduvall@chromium.org <cduvall@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-07 03:18:45 +0000 |
|---|---|---|
| committer | cduvall@chromium.org <cduvall@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-07 03:18:45 +0000 |
| commit | de82f9e2035c4259ff38c4a045abe4f549f71a62 (patch) | |
| tree | 9b4fe0113a99557ba6be8805b347588716e8de3c /chrome/browser/ui | |
| parent | c8d4f606b39ba14e5329355d655c3e80b1bde8ec (diff) | |
| download | chromium_src-de82f9e2035c4259ff38c4a045abe4f549f71a62.zip chromium_src-de82f9e2035c4259ff38c4a045abe4f549f71a62.tar.gz chromium_src-de82f9e2035c4259ff38c4a045abe4f549f71a62.tar.bz2 | |
Force browser action icon size to always be 19px
Fixed the browser action bar so that it will render the icon normal size
if an icon is not specified. Also re-worked a test case for dynamically changing
the browser action icon (BrowserActionApiTest.DynamicBrowserAction).
Made the test more robust by checking the pixels of the icon to make sure it is actually
changed. Screenshots of before and after are posted on the bug.
BUG=111301
TEST=Load an extension with a browser action and no icon specified. The icon
now should be the default extension icon.
Review URL: http://codereview.chromium.org/9424024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125314 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui')
| -rw-r--r-- | chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc b/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc index 3c8f616..2db368c 100644 --- a/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc +++ b/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc @@ -5,6 +5,7 @@ #include "chrome/browser/ui/gtk/browser_actions_toolbar_gtk.h" #include <algorithm> +#include <gtk/gtk.h> #include <vector> #include "base/bind.h" @@ -102,6 +103,7 @@ class BrowserActionButton : public content::NotificationObserver, IDR_BROWSER_ACTION_H, 0, NULL)); + gtk_widget_set_size_request(button(), kButtonWidth, kButtonWidth); alignment_.Own(gtk_alignment_new(0, 0, 1, 1)); gtk_container_add(GTK_CONTAINER(alignment_.get()), button()); gtk_widget_show(button()); |