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

Unified Diff: components/sync_driver/generic_change_processor_unittest.cc

Issue 982883002: [Sync] Add size and crc32c to AttachmentId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge with master. Created 5 years, 9 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
« no previous file with comments | « no previous file | sync/api/attachments/attachment.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync_driver/generic_change_processor_unittest.cc
diff --git a/components/sync_driver/generic_change_processor_unittest.cc b/components/sync_driver/generic_change_processor_unittest.cc
index 8d305001c2edbd28a27456cb83770e826c3734b0..bbd99e4298e0cefcdccf71720670c98f6b723628 100644
--- a/components/sync_driver/generic_change_processor_unittest.cc
+++ b/components/sync_driver/generic_change_processor_unittest.cc
@@ -370,8 +370,8 @@ TEST_F(SyncGenericChangeProcessorTest,
pref_specifics->set_name("test");
syncer::AttachmentIdList attachment_ids;
- attachment_ids.push_back(syncer::AttachmentId::Create());
- attachment_ids.push_back(syncer::AttachmentId::Create());
+ attachment_ids.push_back(syncer::AttachmentId::Create(0, 0));
+ attachment_ids.push_back(syncer::AttachmentId::Create(0, 0));
// Add a SyncData with two attachments.
syncer::SyncChangeList change_list;
@@ -394,7 +394,7 @@ TEST_F(SyncGenericChangeProcessorTest,
// Update the SyncData, replacing its two attachments with one new attachment.
syncer::AttachmentIdList new_attachment_ids;
- new_attachment_ids.push_back(syncer::AttachmentId::Create());
+ new_attachment_ids.push_back(syncer::AttachmentId::Create(0, 0));
mock_attachment_service()->attachment_id_sets()->clear();
change_list.clear();
change_list.push_back(
@@ -424,7 +424,7 @@ TEST_F(SyncGenericChangeProcessorTest, AttachmentUploaded) {
pref_specifics->set_name("test");
syncer::AttachmentIdList attachment_ids;
- attachment_ids.push_back(syncer::AttachmentId::Create());
+ attachment_ids.push_back(syncer::AttachmentId::Create(0, 0));
// Add a SyncData with two attachments.
syncer::SyncChangeList change_list;
@@ -452,8 +452,8 @@ TEST_F(SyncGenericChangeProcessorTest, AttachmentUploaded) {
// scheduled for upload.
TEST_F(SyncGenericChangeProcessorTest, UploadAllAttachmentsNotOnServer) {
// Create two attachment ids. id2 will be marked as "on server".
- syncer::AttachmentId id1 = syncer::AttachmentId::Create();
- syncer::AttachmentId id2 = syncer::AttachmentId::Create();
+ syncer::AttachmentId id1 = syncer::AttachmentId::Create(0, 0);
+ syncer::AttachmentId id2 = syncer::AttachmentId::Create(0, 0);
{
// Write an entry containing these two attachment ids.
syncer::WriteTransaction trans(FROM_HERE, user_share());
« no previous file with comments | « no previous file | sync/api/attachments/attachment.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698