diff options
author | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-17 01:03:26 +0000 |
---|---|---|
committer | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-17 01:03:26 +0000 |
commit | 4c4f8199de84c55ae267506eacfd744e8f3e69f7 (patch) | |
tree | ae469a5a8c542d3c0af754953d0178be168a9385 /chrome/common/extensions/extension.h | |
parent | b4efc650ae68817ff33bd4efab83ffb23d706113 (diff) | |
download | chromium_src-4c4f8199de84c55ae267506eacfd744e8f3e69f7.zip chromium_src-4c4f8199de84c55ae267506eacfd744e8f3e69f7.tar.gz chromium_src-4c4f8199de84c55ae267506eacfd744e8f3e69f7.tar.bz2 |
Allow slightly larger browser and page action icons.
This allows 19px icons to be able to be used for both
browser and page icons.
I think it looks nicer with the page actions slightly smaller, and that is what we also usually do in Chrome,
but some Chrome location bar icons use 18px for soft edges,
so I guess this will just have to be something we advise
developers on.
We can actually fit up to 21 (whoa nelly) pixels on Windows,
but apparently the space is slightly smaller on mac.
Also minor layout fix. We were sizing the browser action
buttons 1px too short.
BUG=24881
TEST=Load chrome/test/data/extensions/samples/icon_size_test. Icons should be 17px for the page action and 19px for the browser action and centered nicely in the space.
Review URL: http://codereview.chromium.org/286001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29365 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common/extensions/extension.h')
-rw-r--r-- | chrome/common/extensions/extension.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/chrome/common/extensions/extension.h b/chrome/common/extensions/extension.h index c96ec0a..f506ffd 100644 --- a/chrome/common/extensions/extension.h +++ b/chrome/common/extensions/extension.h @@ -65,6 +65,10 @@ class Extension { // Icon sizes used by the extension system. static const int kIconSizes[]; + // Max size (both dimensions) for browser and page actions. + static const int kPageActionIconMaxSize; + static const int kBrowserActionIconMaxSize; + // Each permission is a module that the extension is permitted to use. static const char* kPermissionNames[]; static const size_t kNumPermissions; |