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..f3c802320f7b0a0733aa88c532ccf19d6e19c29c 100644
--- a/chrome/browser/search/search.cc
+++ b/chrome/browser/search/search.cc
@@ -362,6 +362,17 @@ uint64 EmbeddedSearchPageVersion() {
   return kEmbeddedPageVersionDefault;
 }
 
+std::string InstantExtendedEnabledParam(bool for_search) {
+#if !defined(OS_IOS) && !defined(OS_ANDROID)

[samarth, 2014/02/04 17:17:05]

Why are these ifdefs necessary?  IsQueryExtractionEnabled already returns true
for mobile.

[kmadhusu, 2014/02/04 17:49:59]

ifdefs are not required here. Since I passed "false" in
toolbar_model_android.cc, I thought it would be better to add ifdefs here so
that I can revisit the code and update later.

[kmadhusu, 2014/02/04 19:39:54]

Removed.

+  // On mobile, query extraction is always enabled.
+  if (for_search && !chrome::IsQueryExtractionEnabled())
+    return std::string();
+#endif
+
+  return std::string(google_util::kInstantExtendedAPIParam) + "=" +
+      base::Uint64ToString(EmbeddedSearchPageVersion()) + "&";
+}
+
 bool IsQueryExtractionEnabled() {
 #if defined(OS_IOS) || defined(OS_ANDROID)
   return true;