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

Unified Diff: chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc

Issue 12556002: Always Close the Autofill UI through the same path (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Disable Test On Linux 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/autofill/autofill_popup_view_gtk.cc
diff --git a/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc b/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc
index 689b639b9d33c14207b2d2aafbe62a604a265d50..1c90b65ccfecec58412ab12bbe2c6d63c4b470fa 100644
--- a/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc
+++ b/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.cc
@@ -65,12 +65,13 @@ AutofillPopupViewGtk::AutofillPopupViewGtk(
}
AutofillPopupViewGtk::~AutofillPopupViewGtk() {
- controller_->ViewDestroyed();
g_object_unref(layout_);
gtk_widget_destroy(window_);
}
void AutofillPopupViewGtk::Hide() {
+ AutofillPopupView::Hide();
+
delete this;
}
@@ -106,7 +107,7 @@ void AutofillPopupViewGtk::UpdateBoundsAndRedrawPopup() {
gboolean AutofillPopupViewGtk::HandleConfigure(GtkWidget* widget,
GdkEventConfigure* event) {
- Hide();
+ controller_->Hide();
return FALSE;
}

Powered by Google App Engine
This is Rietveld 408576698