diff options
| author | erg@chromium.org <erg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-24 20:09:04 +0000 |
|---|---|---|
| committer | erg@chromium.org <erg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-24 20:09:04 +0000 |
| commit | 86e1db9d9de1ce72ba045bd3755e7a077a1f7fd8 (patch) | |
| tree | 4cb97b990b36fbf59a48c15b838a636fcd56dc7e /base/message_loop.h | |
| parent | d4eb69b4e775c1f951120514de8036e7f495374f (diff) | |
| download | chromium_src-86e1db9d9de1ce72ba045bd3755e7a077a1f7fd8.zip chromium_src-86e1db9d9de1ce72ba045bd3755e7a077a1f7fd8.tar.gz chromium_src-86e1db9d9de1ce72ba045bd3755e7a077a1f7fd8.tar.bz2 | |
GTK: Remove the global bookmarks menu by default. Currently, it can't be implemented efficiently.
We suspect that Ubuntu's appmenu code can not efficiently handle hundreds or
thousands of menu items, especially when we both initially push the default
16x16 favicon, and then push the asynchronously loaded favicon. We don't
believe this is a problem in chrome's code since this issue only manifests
itself on Natty, and stack traces in bug #86715 are stuck waiting on dbus in
ubuntu's dbusmenu-glib code.
Puts the bookmark menu behind a flag so other people downstream can debug what's going on. Also reduces the number of iconed menu items to 16, ensuring that at most 16k of favicon data is sent over dbus. (The menus felt overly cluttered previously, anyway.)
BUG=86715,85466
TEST=Have hundreds (thousands?) of bookmarks in your profile. chrome startup shouldn't take minutes on Ubuntu Natty on a desktop that uses appmenu-gtk.
Review URL: http://codereview.chromium.org/7252001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90417 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/message_loop.h')
0 files changed, 0 insertions, 0 deletions