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

Unified Diff: chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc

Issue 99218: The "size-request" event is run-first, clean up the Omnibox handler. (Closed)
Patch Set: Created 11 years, 8 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
« no previous file with comments | « chrome/browser/autocomplete/autocomplete_edit_view_gtk.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc
diff --git a/chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc b/chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc
index 20609db730602fdac392355fcb52af0c7bb86d7f..566c8d2bc263e72940674514472bcaa5f3093188 100644
--- a/chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc
+++ b/chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc
@@ -458,11 +458,10 @@ void AutocompleteEditViewGtk::HandleViewMoveCursor(
g_signal_stop_emission_by_name(text_view_, "move-cursor");
}
-gboolean AutocompleteEditViewGtk::HandleViewSizeRequest(GtkRequisition* req) {
- // Don't force a minimum width, but use the font-relative height.
- GTK_WIDGET_GET_CLASS(text_view_)->size_request(text_view_, req);
+void AutocompleteEditViewGtk::HandleViewSizeRequest(GtkRequisition* req) {
+ // Don't force a minimum width, but use the font-relative height. This is a
+ // run-first handler, so the default handler was already called.
req->width = 1;
- return TRUE; // We already called the default handler.
}
AutocompleteEditViewGtk::CharRange AutocompleteEditViewGtk::GetSelection() {
« no previous file with comments | « chrome/browser/autocomplete/autocomplete_edit_view_gtk.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698