Fix accel leak in task manager and bookmark manager by using

chrome/browser/gtk/task_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/task_manager_gtk.h"
 
 #include <gdk/gdkkeysyms.h>
 
 #include <algorithm>
 #include <vector>
@@ skipping 231 matching lines @@
     process_count_(0) {
   Init();
 }
 
 // static
 TaskManagerGtk* TaskManagerGtk::instance_ = NULL;
 
 TaskManagerGtk::~TaskManagerGtk() {
   task_manager_->OnWindowClosed();
   model_->RemoveObserver(this);
+
+  gtk_accel_group_disconnect_key(accel_group_, GDK_w, GDK_CONTROL_MASK);
+  gtk_window_remove_accel_group(GTK_WINDOW(dialog_), accel_group_);
+  g_object_unref(accel_group_);
+  accel_group_ = NULL;
+
+  gtk_widget_destroy(dialog_);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // TaskManagerGtk, TaskManagerModelObserver implementation:
 
 void TaskManagerGtk::OnModelChanged() {
   // Nothing to do.
 }
 
 void TaskManagerGtk::OnItemsChanged(int start, int length) {
@@ skipping 162 matching lines @@
     }
   }
 
   // Otherwise, just set a default size (GTK will override this if it's not
   // large enough to hold the window's contents).
   gtk_window_set_default_size(
       GTK_WINDOW(dialog_), kDefaultWidth, kDefaultHeight);
 }
 
 void TaskManagerGtk::ConnectAccelerators() {
-  GtkAccelGroup* accel_group = gtk_accel_group_new();
-  gtk_window_add_accel_group(GTK_WINDOW(dialog_), accel_group);
+  accel_group_ = gtk_accel_group_new();
+  gtk_window_add_accel_group(GTK_WINDOW(dialog_), accel_group_);
 
-  // Drop the initial ref on |accel_group| so |dialog_| 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));
 }
 
 void TaskManagerGtk::CreateTaskManagerTreeview() {
   treeview_ = gtk_tree_view_new();
 
   TreeViewInsertColumnWithPixbuf(treeview_, IDS_TASK_MANAGER_PAGE_COLUMN);
   TreeViewInsertColumn(treeview_, IDS_TASK_MANAGER_PHYSICAL_MEM_COLUMN);
@@ skipping 259 matching lines @@
 gboolean TaskManagerGtk::OnGtkAccelerator(GtkAccelGroup* accel_group,
                                           GObject* acceleratable,
                                           guint keyval,
                                           GdkModifierType modifier,
                                           TaskManagerGtk* task_manager) {
   if (keyval == GDK_w && modifier == GDK_CONTROL_MASK) {
     // The GTK_RESPONSE_DELETE_EVENT response must be sent before the widget
     // is destroyed.  The deleted object will receive gtk signals otherwise.
     gtk_dialog_response(GTK_DIALOG(task_manager->dialog_),
                         GTK_RESPONSE_DELETE_EVENT);
-    gtk_widget_destroy(task_manager->dialog_);
   }
 
   return TRUE;
 }