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

Unified Diff: chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk.cc

Issue 12851002: Remove ConstrainedWindowGtkDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move SetBackgroundColor() 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/extensions/media_galleries_dialog_gtk.cc
diff --git a/chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk.cc b/chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk.cc
index abbb1b9456b3e15e2efc04e4e7bb173f6624c7e9..a49609c375a2bff04d122ead01e71472920682b1 100644
--- a/chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk.cc
+++ b/chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk.cc
@@ -30,7 +30,9 @@ MediaGalleriesDialogGtk::MediaGalleriesDialogGtk(
// May be NULL during tests.
if (controller->web_contents()) {
- window_ = CreateWebContentsModalDialogGtk(controller->web_contents(), this);
+ window_ = CreateWebContentsModalDialogGtk(controller->web_contents(),
+ contents_.get(),
+ confirm_);
WebContentsModalDialogManager* web_contents_modal_dialog_manager =
WebContentsModalDialogManager::FromWebContents(
@@ -44,6 +46,10 @@ MediaGalleriesDialogGtk::~MediaGalleriesDialogGtk() {
void MediaGalleriesDialogGtk::InitWidgets() {
contents_.Own(gtk_vbox_new(FALSE, ui::kContentAreaSpacing));
+ g_signal_connect(contents_.get(),
+ "destroy",
+ G_CALLBACK(OnDestroyThunk),
+ this);
GtkWidget* header =
gtk_util::CreateBoldLabel(UTF16ToUTF8(controller_->GetHeader()));
@@ -135,18 +141,6 @@ void MediaGalleriesDialogGtk::ForgetGallery(
checkbox_map_.erase(iter);
}
-GtkWidget* MediaGalleriesDialogGtk::GetWidgetRoot() {
- return contents_.get();
-}
-
-GtkWidget* MediaGalleriesDialogGtk::GetFocusWidget() {
- return confirm_;
-}
-
-void MediaGalleriesDialogGtk::DeleteDelegate() {
- controller_->DialogFinished(accepted_);
-}
-
void MediaGalleriesDialogGtk::OnToggled(GtkWidget* widget) {
if (confirm_)
gtk_widget_set_sensitive(confirm_, TRUE);
@@ -180,6 +174,10 @@ void MediaGalleriesDialogGtk::OnCancel(GtkWidget* widget) {
gtk_widget_destroy(window_);
}
+void MediaGalleriesDialogGtk::OnDestroy(GtkWidget* widget) {
+ controller_->DialogFinished(accepted_);
+}
+
// MediaGalleriesDialogController ----------------------------------------------
// static
« no previous file with comments | « chrome/browser/ui/gtk/extensions/media_galleries_dialog_gtk.h ('k') | chrome/browser/ui/gtk/login_prompt_gtk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698