Chromium Code Reviews| Index: chrome/browser/ui/omnibox/omnibox_edit_model.h |
| diff --git a/chrome/browser/ui/omnibox/omnibox_edit_model.h b/chrome/browser/ui/omnibox/omnibox_edit_model.h |
| index 1ca4635b63896cd3ed536813eb7dc1c53c201e14..562d21997b9bbb93602e9475d6e3d8319160428c 100644 |
| --- a/chrome/browser/ui/omnibox/omnibox_edit_model.h |
| +++ b/chrome/browser/ui/omnibox/omnibox_edit_model.h |
| @@ -507,11 +507,19 @@ class OmniboxEditModel : public AutocompleteControllerDelegate { |
| GURL original_url_; |
| // True if Instant set the current temporary text, as opposed to it being set |
| - // due to the user arrowing up/down through the popup. |
| + // due to the user arrowing up/down through the popup. This can only be true |
| + // if |has_temporary_text_| is true. |
| // TODO(sreeram): This is a temporary hack. Remove it once the omnibox edit |
| // model/view code is decoupled from Instant (among other things). |
| bool is_temporary_text_set_by_instant_; |
| + // True if the current temporary text set by Instant is a search query; false |
| + // if it is a URL that can be directly navigated to. This is only valid if |
| + // |is_temporary_text_set_by_instant_| is true. This field is needed because |
| + // Instant's temporary text doesn't come from the popup model, so we can't |
| + // lookup its type from the current match. |
| + bool is_instant_temporary_text_a_search_query_; |
|
Peter Kasting
2013/04/08 20:13:50
We now have several bools about what the text in t
sreeram
2013/04/08 20:20:48
Can I punt on this? @beaudoin's refactor should ta
Peter Kasting
2013/04/08 20:34:06
Sure.
|
| + |
| // When the user's last action was to paste, we disallow inline autocomplete |
| // (on the theory that the user is trying to paste in a new URL or part of |
| // one, and in either case inline autocomplete would get in the way). |