Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6972)

Unified Diff: chrome/browser/search_engines/template_url_service_unittest.cc

Issue 229763005: Store default search provider data in dictionary pref. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: just pref changes Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/search_engines/template_url_service_unittest.cc
diff --git a/chrome/browser/search_engines/template_url_service_unittest.cc b/chrome/browser/search_engines/template_url_service_unittest.cc
index 91c612c2150165266e13d2bfe457cd9629b58b0c..a5052161a112df7c5ad0108d3b6167fcd409ff6b 100644
--- a/chrome/browser/search_engines/template_url_service_unittest.cc
+++ b/chrome/browser/search_engines/template_url_service_unittest.cc
@@ -1683,3 +1683,19 @@ TEST_F(TemplateURLServiceTest, ExtensionEngineVsPolicy) {
actual_managed_default = model()->GetDefaultSearchProvider();
ExpectSimilar(expected_managed_default.get(), actual_managed_default);
}
+
+TEST_F(TemplateURLServiceTest, PrefSaving) {
+ // Add a new TemplateURL.
+ test_util_.VerifyLoad();
+ TemplateURL* t_url = AddKeywordWithDate(
+ "name1", "key1", "http://foo1/{searchTerms}", "http://sugg1",
+ std::string(), "http://icon1", true, "UTF-8;UTF-16", Time(), Time());
+ model()->SetUserSelectedDefaultSearchProvider(t_url);
+
+ ASSERT_EQ(t_url, model()->GetDefaultSearchProvider());
+ model()->WriteDataToPrefsDictionary(t_url);
+ scoped_ptr<TemplateURL> cloned_url(new TemplateURL(t_url->profile(),
+ t_url->data()));
+ model()->ReadDataFromPrefsDictionary(cloned_url.get());
+ AssertEquals(*cloned_url.get(), *model()->GetDefaultSearchProvider());
+}

Powered by Google App Engine
This is Rietveld 408576698