Adds InstantExtendedEnabledParam to the search request URL iff the query extraction flag is enabled.

chrome/browser/search/search.cc

Index: chrome/browser/search/search.cc
diff --git a/chrome/browser/search/search.cc b/chrome/browser/search/search.cc
index 1562d527433f530e9a6c3cfc56476a7ca89db780..d58c4fab5d21071ef8f013e6d0befe96f8ca66ab 100644
--- a/chrome/browser/search/search.cc
+++ b/chrome/browser/search/search.cc
@@ -362,6 +362,18 @@ uint64 EmbeddedSearchPageVersion() {
   return kEmbeddedPageVersionDefault;
 }
 
+std::string InstantExtendedEnabledParam(bool for_search) {
+  if (for_search && !chrome::IsQueryExtractionEnabled())
+    return std::string();
+
+  uint64 instant_extended_api_version = EmbeddedSearchPageVersion();
+  if (instant_extended_api_version) {

[samarth, 2014/02/07 00:25:31]

Hmm I was going to say this should explicitly check version !=
kEmbeddedPageVersionDisabled but that no longer exists. So you don't need this
check after all :)

[kmadhusu, 2014/02/07 00:44:05]

True. Current version of EmbeddedSearchPageVersion() doesn't return
kEmbeddedPageVersionDisabled but it's better to have this check here to handle
"espv:0" field trials flag.

[samarth, 2014/02/07 17:50:00]

espv:0 is explicitly not a supported config anymore.

[kmadhusu, 2014/02/07 21:25:39]

Removed the check.

+    return std::string(google_util::kInstantExtendedAPIParam) + "=" +
+        base::Uint64ToString(instant_extended_api_version) + "&";
+  }
+  return std::string();
+}
+
 bool IsQueryExtractionEnabled() {
 #if defined(OS_IOS) || defined(OS_ANDROID)
   return true;