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

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

Issue 6627038: gtk: Rename OnDialogResponse() to OnResponse() to standardize on a consistent naming scheme. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 10 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/process_singleton_dialog.cc
diff --git a/chrome/browser/ui/gtk/process_singleton_dialog.cc b/chrome/browser/ui/gtk/process_singleton_dialog.cc
index 78d91cfb10dd97b7949c0e93962dc00952ca8bfc..53f4db07b363cfc79b2e2f4d562c1bfa297cfd99 100644
--- a/chrome/browser/ui/gtk/process_singleton_dialog.cc
+++ b/chrome/browser/ui/gtk/process_singleton_dialog.cc
@@ -4,6 +4,8 @@
#include "chrome/browser/ui/gtk/process_singleton_dialog.h"
+#include <gtk/gtk.h>
+
#include "base/message_loop.h"
#include "chrome/browser/ui/gtk/gtk_util.h"
#include "grit/chromium_strings.h"
@@ -28,15 +30,13 @@ ProcessSingletonDialog::ProcessSingletonDialog(const std::string& message) {
gtk_dialog_add_button(GTK_DIALOG(dialog_), GTK_STOCK_QUIT,
GTK_RESPONSE_REJECT);
- g_signal_connect(dialog_, "response", G_CALLBACK(OnResponse), this);
+ g_signal_connect(dialog_, "response", G_CALLBACK(OnResponseThunk), this);
gtk_widget_show_all(dialog_);
MessageLoop::current()->Run();
}
-// static
-void ProcessSingletonDialog::OnResponse(GtkWidget* widget, int response,
- ProcessSingletonDialog* dialog) {
- gtk_widget_destroy(dialog->dialog_);
+void ProcessSingletonDialog::OnResponse(GtkWidget* dialog, int response_id) {
+ gtk_widget_destroy(dialog_);
MessageLoop::current()->Quit();
}

Powered by Google App Engine
This is Rietveld 408576698