Fix accel leak in task manager and bookmark manager by using

chrome/browser/gtk/bookmark_manager_gtk.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/gtk/bookmark_manager_gtk.h"
 
 #include <gdk/gdkkeysyms.h>
 #include <vector>
 
 #include "app/gtk_dnd_util.h"
@@ skipping 304 matching lines @@
 
   gtk_widget_show_all(window_);
 }
 
 BookmarkManagerGtk::~BookmarkManagerGtk() {
   g_browser_process->local_state()->SetInteger(
       prefs::kBookmarkManagerSplitLocation,
       gtk_paned_get_position(GTK_PANED(paned_)));
   SaveColumnConfiguration();
   model_->RemoveObserver(this);
+
+  gtk_accel_group_disconnect_key(accel_group_, GDK_w, GDK_CONTROL_MASK);
+  gtk_window_remove_accel_group(GTK_WINDOW(window_), accel_group_);
+  g_object_unref(accel_group_);
 }
 
 void BookmarkManagerGtk::InitWidgets() {
   window_ = gtk_window_new(GTK_WINDOW_TOPLEVEL);
   gtk_window_set_title(GTK_WINDOW(window_),
       l10n_util::GetStringUTF8(IDS_BOOKMARK_MANAGER_TITLE).c_str());
   g_signal_connect(
       window_, "configure-event", G_CALLBACK(OnWindowConfiguredThunk), this);
   g_signal_connect(
       window_, "destroy", G_CALLBACK(OnWindowDestroyedThunk), this);
@@ skipping 74 matching lines @@
   }
   gtk_paned_set_position(GTK_PANED(paned_), split_x);
 
   GtkWidget* vbox = gtk_vbox_new(FALSE, 0);
   gtk_box_pack_start(GTK_BOX(vbox), hbox, FALSE, FALSE, 0);
   gtk_box_pack_start(GTK_BOX(vbox), paned_, TRUE, TRUE, 0);
   gtk_container_add(GTK_CONTAINER(window_), vbox);
 }
 
 void BookmarkManagerGtk::ConnectAccelerators() {
-  GtkAccelGroup* accel_group = gtk_accel_group_new();
-  gtk_window_add_accel_group(GTK_WINDOW(window_), accel_group);
+  accel_group_ = gtk_accel_group_new();
+  gtk_window_add_accel_group(GTK_WINDOW(window_), accel_group_);
 
-  // Drop the initial ref on |accel_group| so |window_| will own it.
-  g_object_unref(accel_group);
-
-  gtk_accel_group_connect(accel_group,
+  gtk_accel_group_connect(accel_group_,
                           GDK_w, GDK_CONTROL_MASK, GtkAccelFlags(0),
                           g_cclosure_new(G_CALLBACK(OnGtkAccelerator),
                                          this, NULL));
 }
 
 GtkWidget* BookmarkManagerGtk::MakeLeftPane() {
   left_store_ = bookmark_utils::MakeFolderTreeStore();
   left_tree_view_ = bookmark_utils::MakeTreeViewForStore(left_store_);
 
   // When a row is collapsed that contained the selected node, we want to select
@@ skipping 979 matching lines @@
       modifier & gtk_accelerator_get_default_mod_mask());
   // The only accelerator we have registered is ctrl+w, so any other value is a
   // non-fatal error.
   DCHECK_EQ(keyval, static_cast<guint>(GDK_w));
   DCHECK_EQ(modifier, GDK_CONTROL_MASK);
 
   gtk_widget_destroy(bookmark_manager->window_);
 
   return TRUE;
 }