Instant API: tell page whether the browser is capturing key strokes.

chrome/browser/instant/instant_controller.h

Index: chrome/browser/instant/instant_controller.h
diff --git a/chrome/browser/instant/instant_controller.h b/chrome/browser/instant/instant_controller.h
index f3448cd557667c6f429a69a8d7c9578ee9a5356c..01894f1fd0465c5f854b52eed02cf28ec14a45aa 100644
--- a/chrome/browser/instant/instant_controller.h
+++ b/chrome/browser/instant/instant_controller.h
@@ -86,7 +86,7 @@ class InstantController {
 
   // The omnibox has gained focus. Preload the default search engine, in
   // anticipation of the user typing a query.
-  void OmniboxGotFocus();
+  void OmniboxGotFocus(bool focus_is_visible);
 
   // The search mode in the active tab has changed. Pass the message down to
   // the loader which will notify the renderer.
@@ -223,6 +223,10 @@ class InstantController {
   // True if the omnibox is focused, false otherwise.
   bool is_omnibox_focused_;
 
+  // True iff the omnibox is invisibly focused. Never true if is_omnibox_focused
+  // is false.
+  bool is_omnibox_invisibly_focused_;

[Jered, 2012/11/28 17:50:16]

Would it make sense to have a single enum here, like OmniboxFocusState { NONE,
INVISIBLE, FOCUS }?

[samarth, 2012/11/28 19:33:09]

We could do that.  The downside is that just checking if the omnibox has focus
becomes complicated (need to check two values).  But that's fine by me if you
prefer that.

[Jered, 2012/11/28 19:56:57]

I think we could just say: omnibox_focus_type_ != NONE, right? This is mostly
aesthetic, I mostly prefer it rather than introducing an "impossible" value.

[samarth, 2012/11/28 22:08:11]

Duh, you're right.  Done.

+
   // The search model mode for the active tab.
   chrome::search::Mode search_mode_;