Omnibox: Allow Bundled Omnibox Field Trial to Examine Instant Extended

chrome/browser/omnibox/omnibox_field_trial.h

Index: chrome/browser/omnibox/omnibox_field_trial.h
diff --git a/chrome/browser/omnibox/omnibox_field_trial.h b/chrome/browser/omnibox/omnibox_field_trial.h
index 7bfd2d83eeac28cf166be11879a0180206cc7405..bb1cc4db4047b723831df0d4d9a5e46ea42daeb5 100644
--- a/chrome/browser/omnibox/omnibox_field_trial.h
+++ b/chrome/browser/omnibox/omnibox_field_trial.h
@@ -148,14 +148,13 @@ class OmniboxFieldTrial {
   // prevent search history matches from inlining.
   //
   // This function returns the value associated with the |rule| that applies
-  // in the current context (which currently only consists of
-  // |page_classification| but will soon contain other features, some not
-  // passed in as parameters, such as whether Instant Extended is enabled).
-  // If no such rule exists in the current context, looks for that rule in
-  // the global context and return its value if found.  If the rule remains
-  // unfound in the global context, returns the empty string.  For more
-  // details, see the implementation.  How to interpret the value is left
-  // to the caller; this is rule-dependent.
+  // in the current context (which currently consists of |page_classification|
+  // and whether Instant Extended is enabled).  If no such rule exists in the
+  // current context, backoff to the rule in various wildcard contexts and

[Peter Kasting, 2013/08/09 20:45:45]

Nit; backoff to the rule -> fall back to any rules ... return their values ?

[Mark P, 2013/08/09 21:55:01]

Changed backoff to fall back.  Left the singular rule because rule is specific
(SearchHistory, ReorderForInlining, etc.), and it doesn't fall back to any rule,
it falls back to that rule in different contexts.

+  // return its value if found.  If the rule remains unfound in the global
+  // context, returns the empty string.  For more details, see the

[Peter Kasting, 2013/08/09 20:45:45]

Nit: Perhaps "For more details, including how we prioritize different wildcard
contexts, ..."?

[Mark P, 2013/08/09 21:55:01]

Done.

+  // implementation.  How to interpret the value is left to the caller; this
+  // is rule-dependent.
   static std::string GetValueForRuleInContext(
       const std::string& rule,
       AutocompleteInput::PageClassification page_classification);