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

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

Issue 7396024: Adding a sync_guid field to TemplateURL. Adding appropriate database migration changes and tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Lint fix. Created 9 years, 5 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
===================================================================
--- chrome/browser/search_engines/template_url_service_unittest.cc (revision 92870)
+++ chrome/browser/search_engines/template_url_service_unittest.cc (working copy)
@@ -166,6 +166,7 @@
ASSERT_EQ(expected.show_in_default_list(), actual.show_in_default_list());
ASSERT_TRUE(expected.date_created() == actual.date_created());
ASSERT_TRUE(expected.last_modified() == actual.last_modified());
+ ASSERT_EQ(expected.sync_guid(), actual.sync_guid());
}
// Checks that the two TemplateURLs are similar. It does not check the id, the
@@ -396,6 +397,7 @@
t_url->SetFaviconURL(favicon_url);
t_url->set_date_created(Time::FromTimeT(100));
t_url->set_last_modified(Time::FromTimeT(100));
+ t_url->set_sync_guid("00000000-0000-0000-0000-000000000001");
t_url->set_safe_for_autoreplace(true);
model()->Add(t_url);
ASSERT_TRUE(model()->CanReplaceKeyword(ASCIIToUTF16("keyword"),
« no previous file with comments | « chrome/browser/search_engines/template_url.h ('k') | chrome/browser/sync/protocol/proto_value_conversions.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698