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 6caf62f542932a94b4d3cbe564454c88cc6574c4..6906774a3d7fd7ede55a4e93805fb5f4a7fe60e7 100644 |
| --- a/chrome/browser/ui/omnibox/omnibox_edit_model.h |
| +++ b/chrome/browser/ui/omnibox/omnibox_edit_model.h |
| @@ -37,13 +37,15 @@ class OmniboxEditModel : public AutocompleteControllerDelegate { |
| State(bool user_input_in_progress, |
| const string16& user_text, |
| const string16& keyword, |
| - bool is_keyword_hint); |
| + bool is_keyword_hint, |
| + bool is_focus_visible); |
| ~State(); |
| bool user_input_in_progress; |
| const string16 user_text; |
| const string16 keyword; |
| const bool is_keyword_hint; |
| + const bool is_focus_visible; |
| }; |
| OmniboxEditModel(OmniboxView* view, |
| @@ -191,6 +193,8 @@ class OmniboxEditModel : public AutocompleteControllerDelegate { |
| bool has_focus() const { return has_focus_; } |
|
Peter Kasting
2012/12/04 02:00:52
Nit: Blank line unnecessary
samarth
2012/12/04 04:55:35
Done.
|
| + bool is_focus_visible() const { return is_focus_visible_; } |
| + |
| // Accessors for keyword-related state (see comments on keyword_ and |
| // is_keyword_hint_). |
| const string16& keyword() const { return keyword_; } |
| @@ -213,6 +217,10 @@ class OmniboxEditModel : public AutocompleteControllerDelegate { |
| // control key is down (at the time we're gaining focus). |
| void OnSetFocus(bool control_down); |
| + // Sets the visibility of omnibox focus. Does not impact whether the omnibox |
| + // currently has focus or not. |
|
Peter Kasting
2012/12/04 02:00:52
Nit: Add note like "This setting will be reset to
samarth
2012/12/04 04:55:35
Done.
|
| + void SetFocusVisibility(bool visible); |
| + |
| // Sent before |OnKillFocus| and before the popup is closed. |
| void OnWillKillFocus(gfx::NativeView view_gaining_focus); |
| @@ -404,6 +412,11 @@ class OmniboxEditModel : public AutocompleteControllerDelegate { |
| // Whether the edit has focus. |
| bool has_focus_; |
| + // Will the focus be visible? Note that this does NOT describe whether the |
| + // omnibox currently has focus, only whether focusing will be visible when it |
| + // is focused. |
| + bool is_focus_visible_; |
| + |
| // The URL of the currently displayed page. |
| string16 permanent_text_; |