Add a preference to control Windows desktop search redirection.

components/search_engines/desktop_search_win.cc

 // Copyright 2015 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 "components/search_engines/detect_desktop_search_win.h"
+#include "components/search_engines/desktop_search_win.h"
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
+#include "base/prefs/pref_service.h"
 #include "base/strings/string_util.h"
+#include "components/pref_registry/pref_registry_syncable.h"
 #include "components/search_engines/prepopulated_engines.h"
 #include "components/search_engines/template_url.h"
 #include "components/search_engines/template_url_prepopulate_data.h"
 #include "net/base/url_util.h"
 
+namespace {
+// Name of the Windows desktop search redirection preference.
+const char kWindowsDesktopSearchRedirectionPref[] =
+    "windows_desktop_search_redirection";

[Peter Kasting, 2015/12/03 05:41:39]

Multiple test files also want to use this pref name.  I suggest declaring it
publicly so the tests can just use this string constant directly and not risk
getting out of sync.

[fdoray, 2015/12/03 17:16:17]

Done.

+}  // namespace
+
+const base::Feature kWindowsDesktopSearchRedirectionFeature = {
+    "WindowsDesktopSearchRedirection", base::FEATURE_DISABLED_BY_DEFAULT};

[Peter Kasting, 2015/12/03 05:41:39]

Nit: I'd wrap the final "};" to the next line.

[fdoray, 2015/12/03 17:16:18]

Done.

+
+void RegisterWindowsDesktopSearchRedirectionPref(
+    user_prefs::PrefRegistrySyncable* registry) {
+  registry->RegisterBooleanPref(kWindowsDesktopSearchRedirectionPref, false);
+}
+
+bool ShouldRedirectWindowsDesktopSearchToDefaultSearchEngine(
+    PrefService* pref_service) {
+  DCHECK(pref_service);
+
+  if (!base::FeatureList::IsEnabled(kWindowsDesktopSearchRedirectionFeature))
+    return false;
+  return pref_service->GetBoolean(kWindowsDesktopSearchRedirectionPref);

[Peter Kasting, 2015/12/03 05:41:39]

Nit: Simpler:

  return base::FeatureList::IsEnabled(
             kWindowsDesktopSearchRedirectionFeature) &&
         pref_service->GetBoolean(kWindowsDesktopSearchRedirectionPref);

[fdoray, 2015/12/03 17:16:17]

Done.

+}
+
 bool DetectWindowsDesktopSearch(const GURL& url,
                                 const SearchTermsData& search_terms_data,
                                 base::string16* search_terms) {
   DCHECK(search_terms);
 
   scoped_ptr<TemplateURLData> template_url_data =
       TemplateURLPrepopulateData::MakeTemplateURLDataFromPrepopulatedEngine(
           TemplateURLPrepopulateData::bing);
   TemplateURL template_url(*template_url_data);
 
@@ skipping 13 matching lines @@
     if (base::EqualsCaseInsensitiveASCII(it.GetKey(), kBingSourceQueryKey)) {
       std::string source = it.GetValue();
       if (source == kBingSourceDesktopText || source == kBingSourceDesktopVoice)
         return true;
     }
   }
 
   search_terms->clear();
   return false;
 }