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

Unified Diff: chrome/browser/ui/omnibox/omnibox_edit_model.h

Issue 11418144: [Search] Implementation of the invisible focus on Windows (Closed) Base URL: http://git.chromium.org/chromium/src.git@samarthlatest
Patch Set: Rebased to master instead of other change Created 8 years, 1 month 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/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..cd40872d371cedde46d50cb7790487842fcdc3a4 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,
@@ -67,6 +69,10 @@ class OmniboxEditModel : public AutocompleteControllerDelegate {
Profile* profile() const { return profile_; }
+ bool is_focus_visible() { return is_focus_visible_; }
+
+ void set_is_focus_visible(bool value) { is_focus_visible_ = value; }
+
// Returns the current state. This assumes we are switching tabs, and changes
// the internal state appropriately.
const State GetStateForTabSwitch();
@@ -404,6 +410,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_;

Powered by Google App Engine
This is Rietveld 408576698