Make extensions DSE persistent in browser prefs

components/search_engines/search_engines_test_util.h

Index: components/search_engines/search_engines_test_util.h
diff --git a/components/search_engines/search_engines_test_util.h b/components/search_engines/search_engines_test_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..e1c8f47ab1df735535b0c37ab9a03d89a149163f
--- /dev/null
+++ b/components/search_engines/search_engines_test_util.h
@@ -0,0 +1,22 @@
+// Copyright 2016 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 COMPONENTS_SEARCH_ENGINES_SEARCH_ENGINES_TEST_UTIL_H_
+#define COMPONENTS_SEARCH_ENGINES_SEARCH_ENGINES_TEST_UTIL_H_
+
+#include <string>
+
+struct TemplateURLData;
+
+// Generate TemplateUrlData structure useful for tests filled with values

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

Nit: Url -> URL

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

Done

+// autogenerated from provider_name param.
+std::unique_ptr<TemplateURLData> GenerateDummyTemplateURLData(

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

I'm kinda surprised we don't have functions like these two already somewhere.

I'm also wondering if we could refactor more of our tests code (not in this CL!)
to use these.

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

I think I saw several similar functions across code, for example in
template_url_service_unittest.cc.
Refactor is possible.

+    const std::string& provider_name);
+
+// Checks that the two TemplateURLs are similar. Does not check the id, the
+// date_created or the last_modified time.  Neither pointer should be nullptr.

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

Nit: In comments, just say "null".

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

Done

+void ExpectSimilar(const TemplateURLData* expected,
+                   const TemplateURLData* actual);
+
+#endif  // COMPONENTS_SEARCH_ENGINES_SEARCH_ENGINES_TEST_UTIL_H_