Skip to content

Commit

Permalink
[gtk] Fix a widget leak.
Browse files Browse the repository at this point in the history
BUG=none
TEST=manual

Review URL: http://codereview.chromium.org/4835002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65973 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
estade@chromium.org committed Nov 12, 2010
1 parent 94fdd85 commit 312114c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion chrome/browser/gtk/bookmark_menu_controller_gtk.cc
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ BookmarkMenuController::BookmarkMenuController(Browser* browser,
ignore_button_release_(false),
triggering_widget_(NULL) {
menu_ = gtk_menu_new();
g_object_ref_sink(menu_);
BuildMenu(node, start_child_index, menu_);
signals_.Connect(menu_, "hide",
G_CALLBACK(OnMenuHiddenThunk), this);
Expand All @@ -88,7 +89,8 @@ BookmarkMenuController::BookmarkMenuController(Browser* browser,

BookmarkMenuController::~BookmarkMenuController() {
profile_->GetBookmarkModel()->RemoveObserver(this);
gtk_menu_popdown(GTK_MENU(menu_));
gtk_widget_destroy(menu_);
g_object_unref(menu_);
}

void BookmarkMenuController::Popup(GtkWidget* widget, gint button_type,
Expand Down

0 comments on commit 312114c

Please sign in to comment.