diff options
author | erg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-17 17:50:45 +0000 |
---|---|---|
committer | erg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-17 17:50:45 +0000 |
commit | 8bbb3ac22f87e12532a1c74992eba97bb0a52812 (patch) | |
tree | 60e0da57ce409e7fa40e66563627515de751ef90 /chrome/browser/ui/gtk/global_menu_bar.cc | |
parent | fb497ca596afe928e180609c579208db40f56d3d (diff) | |
download | chromium_src-8bbb3ac22f87e12532a1c74992eba97bb0a52812.zip chromium_src-8bbb3ac22f87e12532a1c74992eba97bb0a52812.tar.gz chromium_src-8bbb3ac22f87e12532a1c74992eba97bb0a52812.tar.bz2 |
GTK: Remove the global bookmark menu code. It's been behind a flag and bitrotting.
BUG=86715, 85466
TEST=none
Review URL: http://codereview.chromium.org/8293002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105841 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/gtk/global_menu_bar.cc')
-rw-r--r-- | chrome/browser/ui/gtk/global_menu_bar.cc | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/chrome/browser/ui/gtk/global_menu_bar.cc b/chrome/browser/ui/gtk/global_menu_bar.cc index 4e37ed8..f03a45c 100644 --- a/chrome/browser/ui/gtk/global_menu_bar.cc +++ b/chrome/browser/ui/gtk/global_menu_bar.cc @@ -12,7 +12,6 @@ #include "chrome/browser/profiles/profile.h" #include "chrome/browser/ui/browser.h" #include "chrome/browser/ui/gtk/accelerators_gtk.h" -#include "chrome/browser/ui/gtk/global_bookmark_menu.h" #include "chrome/browser/ui/gtk/gtk_theme_service.h" #include "chrome/browser/ui/gtk/gtk_util.h" #include "chrome/common/chrome_notification_types.h" @@ -118,14 +117,6 @@ GlobalMenuBarCommand history_menu[] = { { MENU_END, MENU_END } }; -GlobalMenuBarCommand bookmark_menu[] = { - { IDS_BOOKMARK_MANAGER, IDC_SHOW_BOOKMARK_MANAGER }, - { IDS_BOOKMARK_CURRENT_PAGE_LINUX, IDC_BOOKMARK_PAGE }, - { IDS_BOOKMARK_ALL_TABS_LINUX, IDC_BOOKMARK_ALL_TABS }, - - { MENU_END, MENU_END } -}; - GlobalMenuBarCommand tools_menu[] = { { IDS_SHOW_DOWNLOADS, IDC_SHOW_DOWNLOADS }, { IDS_SHOW_HISTORY, IDC_SHOW_HISTORY }, @@ -173,22 +164,6 @@ GlobalMenuBar::GlobalMenuBar(Browser* browser) BuildGtkMenuFrom(IDS_HISTORY_MENU_LINUX, &id_to_menu_item_, history_menu, &history_menu_); - if (CommandLine::ForCurrentProcess()->HasSwitch( - switches::kEnableGlobalBookmarkMenu)) { - // TODO(erg): dbusmenu-glib in Ubuntu Natty does not like it when we shove - // 100k or more of favicon data over it. Users have reported that the - // browser hangs on startup for over a minute and breaking during this time - // shows a stack entirely of dbus/glib code (See #86715). For now, just - // hide the menu until appmenu-gtk catches up and doesn't hang if we throw - // (potentially) megs of icon data at it. (Some of our users have thousands - // of bookmarks and we're already unacceptably laggy at 100.) - // - // http://crbug.com/86715, http://crbug.com/85466 - bookmark_menu_.reset(new GlobalBookmarkMenu(browser_)); - BuildGtkMenuFrom(IDS_BOOKMARKS_MENU_LINUX, &id_to_menu_item_, bookmark_menu, - bookmark_menu_.get()); - } - BuildGtkMenuFrom(IDS_TOOLS_MENU_LINUX, &id_to_menu_item_, tools_menu, NULL); BuildGtkMenuFrom(IDS_HELP_MENU_LINUX, &id_to_menu_item_, help_menu, NULL); |