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

chrome/browser/search/search.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/search/search.h"
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
@@ skipping 344 matching lines @@
 uint64 EmbeddedSearchPageVersion() {
   FieldTrialFlags flags;
   if (GetFieldTrialInfo(&flags)) {
     return GetUInt64ValueForFlagWithDefault(kEmbeddedPageVersionFlagName,
                                             kEmbeddedPageVersionDefault,
                                             flags);
   }
   return kEmbeddedPageVersionDefault;
 }
 
+std::string InstantExtendedEnabledParam(bool for_search) {

[samarth, 2014/02/05 21:16:16]

So SearchTermsData checked that EmbeddedSearchPageVersion() was non-zero.
Shouldn't that check be here as well?  (If yes, please also add a test to catch
that case.)

[kmadhusu, 2014/02/05 21:59:40]

Good catch. Fixed. Added test.

+  if (for_search && !chrome::IsQueryExtractionEnabled())
+    return std::string();
+
+  return std::string(google_util::kInstantExtendedAPIParam) + "=" +
+      base::Uint64ToString(EmbeddedSearchPageVersion()) + "&";
+}
+
 bool IsQueryExtractionEnabled() {
 #if defined(OS_IOS) || defined(OS_ANDROID)
   return true;
 #else
   if (!IsInstantExtendedAPIEnabled())
     return false;
 
   const CommandLine* command_line = CommandLine::ForCurrentProcess();
   if (command_line->HasSwitch(switches::kEnableQueryExtraction))
     return true;
@@ skipping 401 matching lines @@
 
 // Given a FieldTrialFlags object, returns the boolean value of the provided
 // flag.
 bool GetBoolValueForFlagWithDefault(const std::string& flag,
                                     bool default_value,
                                     const FieldTrialFlags& flags) {
   return !!GetUInt64ValueForFlagWithDefault(flag, default_value ? 1 : 0, flags);
 }
 
 }  // namespace chrome