Make extensions DSE persistent in browser prefs

chrome/browser/search_engines/template_url_service_test_util.cc

 // Copyright (c) 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_engines/template_url_service_test_util.h"
 
 #include "base/macros.h"
 #include "base/run_loop.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "chrome/browser/history/history_service_factory.h"
@@ skipping 109 matching lines @@
 void TemplateURLServiceTestUtil::ClearModel() {
   model_->Shutdown();
   model_.reset();
   search_terms_data_ = NULL;
 }
 
 void TemplateURLServiceTestUtil::ResetModel(bool verify_load) {
   if (model_)
     ClearModel();
   search_terms_data_ = new TestingSearchTermsData("http://www.google.com/");
+
   model_.reset(new TemplateURLService(
       profile()->GetPrefs(),
       std::unique_ptr<SearchTermsData>(search_terms_data_),
       web_data_service_.get(),
       std::unique_ptr<TemplateURLServiceClient>(
           new TestingTemplateURLServiceClient(
               HistoryServiceFactory::GetForProfileIfExists(
                   profile(), ServiceAccessType::EXPLICIT_ACCESS),
               &search_term_)),
       NULL, NULL, base::Closure()));
   model()->AddObserver(this);
   changed_count_ = 0;
   if (verify_load)
     VerifyLoad();
 }
 
 base::string16 TemplateURLServiceTestUtil::GetAndClearSearchTerm() {
   base::string16 search_term;
   search_term.swap(search_term_);
   return search_term;
 }
 
 void TemplateURLServiceTestUtil::SetGoogleBaseURL(const GURL& base_url) {
   DCHECK(base_url.is_valid());
   search_terms_data_->set_google_base_url(base_url.spec());
   model_->GoogleBaseURLChanged();
 }
+
+void TemplateURLServiceTestUtil::SetExtensionDefaultSearchInPrefs(
+    const std::string& ext_id,
+    const TemplateURLData& ext_data) {
+  std::unique_ptr<base::DictionaryValue> dict =
+      TemplateURLDataToDictionary(ext_data);
+  // Set extension overriden value in test pref service.
+  profile()->GetTestingPrefService()->SetExtensionPref(
+      DefaultSearchManager::kDefaultSearchProviderDataPrefName, dict.release());
+}
+
+void TemplateURLServiceTestUtil::RemoveExtensionDefaultSearchFromPrefs(
+    const std::string& ext_id) {
+  profile()->GetTestingPrefService()->RemoveExtensionPref(
+      DefaultSearchManager::kDefaultSearchProviderDataPrefName);
+}
+
+TemplateURL* TemplateURLServiceTestUtil::AddExtensionControlledTURL(
+    std::unique_ptr<TemplateURL> ext_dse,
+    std::unique_ptr<TemplateURL::AssociatedExtensionInfo> ext_info) {
+  TemplateURL::AssociatedExtensionInfo ext_info_copy = *ext_info;
+  TemplateURLData ext_data = ext_dse->data();
+  TemplateURL* result = model()->AddExtensionControlledTURL(
+      std::move(ext_dse), std::move(ext_info));
+  if (ext_info_copy.wants_to_be_default_engine) {
+    // Set extension overriden DSE value to prefs.

[Peter Kasting, 2016/12/22 20:49:03]

Nit: This comment seems to just restate the function name below.  Is it really
adding anything?

[Alexander Yashkin, 2016/12/23 19:44:08]

Removed

+    SetExtensionDefaultSearchInPrefs(ext_info_copy.extension_id, ext_data);
+  }
+  return result;
+}
+
+void TemplateURLServiceTestUtil::RemoveExtensionControlledTURL(
+    const std::string& ext_id) {
+  TemplateURL* ext_turl = model()->FindTemplateURLForExtension(
+      ext_id, TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION);
+  ASSERT_TRUE(ext_turl);
+  ASSERT_TRUE(ext_turl->GetExtensionInfoForTesting());
+  if (ext_turl->GetExtensionInfoForTesting()->wants_to_be_default_engine) {

[Peter Kasting, 2016/12/22 20:49:03]

Nit: No {}

[Alexander Yashkin, 2016/12/23 19:44:08]

Removed

+    RemoveExtensionDefaultSearchFromPrefs(ext_id);
+  }
+  model()->RemoveExtensionControlledTURL(
+      ext_id, TemplateURL::NORMAL_CONTROLLED_BY_EXTENSION);
+}