Make extensions DSE persistent in browser prefs

chrome/browser/search_engines/template_url_service_test_util.h

 // 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.
 
 #ifndef CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_SERVICE_TEST_UTIL_H_
 #define CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_SERVICE_TEST_UTIL_H_
 
 #include <memory>
 #include <string>
 
 #include "base/files/scoped_temp_dir.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/strings/string16.h"
+#include "components/search_engines/template_url.h"
 #include "components/search_engines/template_url_data.h"
 #include "components/search_engines/template_url_service_observer.h"
 
 class GURL;
 class KeywordWebDataService;
 class TemplateURLService;
 class TestingProfile;
 class TestingSearchTermsData;
 
 // Sets the managed preferences for the default search provider.
@@ skipping 33 matching lines @@
   // Creates a new TemplateURLService.
   void ResetModel(bool verify_load);
 
   // Returns the search term from the last invocation of
   // TemplateURLService::SetKeywordSearchTermsForURL and clears the search term.
   base::string16 GetAndClearSearchTerm();
 
   // Sets the google base url.  |base_url| must be valid.
   void SetGoogleBaseURL(const GURL& base_url);
 
+  // Adds extension controlled TemplateURL to model and overrides default search
+  // pref in extension controlled preferences if extension wants to be default.

[Peter Kasting, 2017/01/06 01:44:57]

Nit: This comment is missing articles ("an", "the", etc.) on a lot of things

[Alexander Yashkin, 2017/01/07 12:55:59]

Tried to fix, hope I get it right.

+  TemplateURL* AddExtensionControlledTURL(
+      std::unique_ptr<TemplateURL> extension_turl,
+      std::unique_ptr<TemplateURL::AssociatedExtensionInfo> info);
+  // Removes a TemplateURL controlled by |extension_id| from the model, and,

[Peter Kasting, 2017/01/06 01:44:57]

Nit: Blank line above this

[Alexander Yashkin, 2017/01/07 12:55:59]

Done.

+  // if necessary, from the extension-controlled default search preference.
+  // This TemplateURL must exist.
+  void RemoveExtensionControlledTURL(const std::string& extension_id);
+
   KeywordWebDataService* web_data_service() { return web_data_service_.get(); }
   TemplateURLService* model() { return model_.get(); }
   TestingProfile* profile() { return profile_.get(); }
 
  private:
   std::unique_ptr<TestingProfile> profile_;
   base::ScopedTempDir temp_dir_;
   int changed_count_;
   base::string16 search_term_;
   scoped_refptr<KeywordWebDataService> web_data_service_;
   TestingSearchTermsData* search_terms_data_;
   std::unique_ptr<TemplateURLService> model_;
 
   DISALLOW_COPY_AND_ASSIGN(TemplateURLServiceTestUtil);
 };
 
 #endif  // CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_SERVICE_TEST_UTIL_H_