Cleanup in components/search and chrome/browser/search

components/search/search.cc

Index: components/search/search.cc
diff --git a/components/search/search.cc b/components/search/search.cc
index 213f2dcc4cf90d819bdb751d87fc6223620a9368..b484b2096f68e6aada3ffb914023a37511bdaae7 100644
--- a/components/search/search.cc
+++ b/components/search/search.cc
@@ -99,40 +99,23 @@ bool GetFieldTrialInfo(FieldTrialFlags* flags) {
   return true;
 }
 
-// Given a FieldTrialFlags object, returns the string value of the provided
-// flag.
-std::string GetStringValueForFlagWithDefault(const std::string& flag,
-                                             const std::string& default_value,
-                                             const FieldTrialFlags& flags) {
-  FieldTrialFlags::const_iterator i;
-  for (i = flags.begin(); i != flags.end(); i++) {
-    if (i->first == flag)
-      return i->second;
-  }
-  return default_value;
-}
-
 // Given a FieldTrialFlags object, returns the uint64_t value of the provided
 // flag.
 uint64_t GetUInt64ValueForFlagWithDefault(const std::string& flag,
                                           uint64_t default_value,
                                           const FieldTrialFlags& flags) {
-  uint64_t value;
-  std::string str_value =
-      GetStringValueForFlagWithDefault(flag, std::string(), flags);
-  if (base::StringToUint64(str_value, &value))
-    return value;
+  FieldTrialFlags::const_iterator i;

[sfiera, 2017/01/24 11:16:49]

Move declaration into loop?

[Marc Treib, 2017/01/24 12:37:22]

I replaced the loop with a for-each. Iterators must die :)

+  for (i = flags.begin(); i != flags.end(); i++) {
+    if (i->first == flag) {
+      const std::string& str_value = i->second;
+      uint64_t value;
+      if (base::StringToUint64(str_value, &value))
+        return value;
+    }
+  }
   return default_value;
 }
 
-// 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);
-}
-
 std::string InstantExtendedEnabledParam() {
   return std::string(google_util::kInstantExtendedAPIParam) + "=" +
          base::Uint64ToString(EmbeddedSearchPageVersion()) + "&";
@@ -143,21 +126,4 @@ std::string ForceInstantResultsParam(bool for_prerender) {
                                                            : std::string();
 }
 
-bool ShouldPrefetchSearchResults() {
-  return IsInstantExtendedAPIEnabled();
-}
-
-bool ShouldReuseInstantSearchBasePage() {
-  return IsInstantExtendedAPIEnabled();
-}
-
-// |url| should either have a secure scheme or have a non-HTTPS base URL that
-// the user specified using --google-base-url. (This allows testers to use
-// --google-base-url to point at non-HTTPS servers, which eases testing.)
-bool IsSuitableURLForInstant(const GURL& url, const TemplateURL* template_url) {
-  return template_url->HasSearchTermsReplacementKey(url) &&
-         (url.SchemeIsCryptographic() ||
-          google_util::StartsWithCommandLineGoogleBaseURL(url));
-}
-
 }  // namespace search