Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3823)

Unified Diff: chrome/browser/ui/gtk/collected_cookies_gtk.cc

Issue 12851002: Remove ConstrainedWindowGtkDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/gtk/collected_cookies_gtk.cc
diff --git a/chrome/browser/ui/gtk/collected_cookies_gtk.cc b/chrome/browser/ui/gtk/collected_cookies_gtk.cc
index d38e2af1090ee4dd3364275e2f52e9da90ac23a3..46de438ebf22968f790945d2cede12b6b4d256ac 100644
--- a/chrome/browser/ui/gtk/collected_cookies_gtk.cc
+++ b/chrome/browser/ui/gtk/collected_cookies_gtk.cc
@@ -121,6 +121,7 @@ CollectedCookiesGtk::CollectedCookiesGtk(GtkWindow* parent,
void CollectedCookiesGtk::Init() {
dialog_ = gtk_vbox_new(FALSE, ui::kContentAreaSpacing);
gtk_box_set_spacing(GTK_BOX(dialog_), ui::kContentAreaSpacing);
+ g_signal_connect(dialog_, "destroy", G_CALLBACK(OnDestroyThunk), this);
GtkWidget* label = gtk_label_new(
l10n_util::GetStringUTF8(IDS_COLLECTED_COOKIES_DIALOG_TITLE).c_str());
@@ -190,7 +191,9 @@ void CollectedCookiesGtk::Init() {
blocked_cookies_tree_adapter_->Init();
EnableControls();
ShowCookieInfo(gtk_notebook_get_current_page(GTK_NOTEBOOK(notebook_)));
- window_ = CreateWebContentsModalDialogGtk(web_contents_, this);
+ window_ = CreateWebContentsModalDialogGtk(web_contents_,
+ dialog_,
+ close_button_);
WebContentsModalDialogManager* web_contents_modal_dialog_manager =
WebContentsModalDialogManager::FromWebContents(web_contents_);
@@ -400,18 +403,6 @@ CollectedCookiesGtk::~CollectedCookiesGtk() {
gtk_widget_destroy(dialog_);
}
-GtkWidget* CollectedCookiesGtk::GetWidgetRoot() {
- return dialog_;
-}
-
-GtkWidget* CollectedCookiesGtk::GetFocusWidget() {
- return close_button_;
-}
-
-void CollectedCookiesGtk::DeleteDelegate() {
- delete this;
-}
-
bool CollectedCookiesGtk::SelectionContainsHostNode(
GtkTreeSelection* selection, gtk_tree::TreeAdapter* adapter) {
// Check whether at least one "origin" node is selected.
@@ -550,3 +541,7 @@ void CollectedCookiesGtk::OnTreeViewSelectionChange(GtkWidget* selection) {
EnableControls();
ShowCookieInfo(gtk_notebook_get_current_page(GTK_NOTEBOOK(notebook_)));
}
+
+void CollectedCookiesGtk::OnDestroy(GtkWidget* widget) {
+ delete this;
+}

Powered by Google App Engine
This is Rietveld 408576698