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

Unified Diff: chrome/browser/ui/gtk/extension_install_prompt_gtk.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/extension_install_prompt_gtk.cc
diff --git a/chrome/browser/ui/gtk/extension_install_prompt_gtk.cc b/chrome/browser/ui/gtk/extension_install_prompt_gtk.cc
index 9790cedc80e2e112a778ff28b43eaaa6c6435041..152dd343b4da52876240ca44749e257d17a9c054 100644
--- a/chrome/browser/ui/gtk/extension_install_prompt_gtk.cc
+++ b/chrome/browser/ui/gtk/extension_install_prompt_gtk.cc
@@ -26,15 +26,15 @@ namespace {
// Left or right margin.
const int kPanelHorizMargin = 13;
-void OnDialogResponse(GtkDialog* dialog, int response_id,
- ExtensionInstallUI::Delegate* delegate) {
+void OnResponse(GtkWidget* dialog, int response_id,
+ ExtensionInstallUI::Delegate* delegate) {
if (response_id == GTK_RESPONSE_ACCEPT) {
delegate->InstallUIProceed();
} else {
delegate->InstallUIAbort();
}
- gtk_widget_destroy(GTK_WIDGET(dialog));
+ gtk_widget_destroy(dialog);
}
void ShowInstallPromptDialog(GtkWindow* parent, SkBitmap* skia_icon,
@@ -79,7 +79,7 @@ void ShowInstallPromptDialog(GtkWindow* parent, SkBitmap* skia_icon,
gtk_misc_set_alignment(GTK_MISC(heading_label), 0.0, 0.5);
gtk_box_pack_start(GTK_BOX(right_column_area), heading_label, TRUE, TRUE, 0);
- g_signal_connect(dialog, "response", G_CALLBACK(OnDialogResponse), delegate);
+ g_signal_connect(dialog, "response", G_CALLBACK(OnResponse), delegate);
gtk_window_set_resizable(GTK_WINDOW(dialog), FALSE);
gtk_widget_show_all(dialog);
}

Powered by Google App Engine
This is Rietveld 408576698