Chromium Code Reviews| Index: chrome/browser/gtk/location_bar_view_gtk.h |
| diff --git a/chrome/browser/gtk/location_bar_view_gtk.h b/chrome/browser/gtk/location_bar_view_gtk.h |
| index 67c8fdcdebda44167c03685c4ff392ecdd13f4a8..a2a32089430c633b57dada16fd0d3980ba79b921 100644 |
| --- a/chrome/browser/gtk/location_bar_view_gtk.h |
| +++ b/chrome/browser/gtk/location_bar_view_gtk.h |
| @@ -33,9 +33,8 @@ class LocationBarViewGtk : public AutocompleteEditController, |
| void SetProfile(Profile* profile); |
| - // Return the native vbox widget. You must call Init() first, or the result |
| - // will be NULL. This is the widget that an embedder should host. |
| - GtkWidget* widget() { return vbox_; } |
| + // Returns the widget the caller should host. You must call Init() first. |
| + GtkWidget* widget() { return outer_bin_; } |
| // Updates the location bar. We also reset the bar's permanent text and |
| // security style, and, if |tab_for_state_restoring| is non-NULL, also |
| @@ -63,7 +62,12 @@ class LocationBarViewGtk : public AutocompleteEditController, |
| virtual void SaveStateToContents(TabContents* contents); |
| private: |
| - GtkWidget* vbox_; |
| + // The outermost widget we want to be hosted. |
| + GtkWidget* outer_bin_; |
| + |
| + // This is the widget you probably care about, our inner vbox (inside the |
|
Evan Martin
2009/03/13 16:33:29
Who is "you" in this case? The first half of this
|
| + // the border) which holds the elements inside the location bar. |
| + GtkWidget* inner_vbox_; |
| scoped_ptr<AutocompleteEditViewGtk> location_entry_; |