diff options
author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-07 23:02:35 +0000 |
---|---|---|
committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-07 23:02:35 +0000 |
commit | e37397a176eac68e8b18b2708e4ee41e131af0bf (patch) | |
tree | 830f73f10c93153320c2bb1ffda29bc36bdbe723 /chrome/browser/views/bookmark_menu_controller_views.cc | |
parent | 3766238379caee23c192b3d1896f3cc76e0d86b4 (diff) | |
download | chromium_src-e37397a176eac68e8b18b2708e4ee41e131af0bf.zip chromium_src-e37397a176eac68e8b18b2708e4ee41e131af0bf.tar.gz chromium_src-e37397a176eac68e8b18b2708e4ee41e131af0bf.tar.bz2 |
Changes the max width for bookmark menus back to 400 and leaves them
at 800 for the wrench menu. Apparently folks don't like wide bookmark
menus.
BUG=50653
TEST=see bug
Review URL: http://codereview.chromium.org/3317011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58771 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/bookmark_menu_controller_views.cc')
-rw-r--r-- | chrome/browser/views/bookmark_menu_controller_views.cc | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/chrome/browser/views/bookmark_menu_controller_views.cc b/chrome/browser/views/bookmark_menu_controller_views.cc index dfc500d..c86432e 100644 --- a/chrome/browser/views/bookmark_menu_controller_views.cc +++ b/chrome/browser/views/bookmark_menu_controller_views.cc @@ -24,6 +24,10 @@ using views::MenuItemView; +// Max width of a menu. There does not appear to be an OS value for this, yet +// both IE and FF restrict the max width of a menu. +static const int kMaxMenuWidth = 400; + BookmarkMenuController::BookmarkMenuController(Browser* browser, Profile* profile, PageNavigator* navigator, @@ -258,6 +262,10 @@ views::MenuItemView* BookmarkMenuController::GetSiblingMenu( return alt_menu; } +int BookmarkMenuController::GetMaxWidthForMenu() { + return kMaxMenuWidth; +} + void BookmarkMenuController::BookmarkModelChanged() { menu_->Cancel(); } |