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

Unified Diff: components/sync/core/processor_entity_tracker_unittest.cc

Issue 2350803005: [Sync] Fixing two bugs in the worker revealed by trying to add an encryption integration test. (Closed)
Patch Set: Merging conflict. Created 4 years, 3 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 | components/sync/engine_impl/model_type_worker.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/core/processor_entity_tracker_unittest.cc
diff --git a/components/sync/core/processor_entity_tracker_unittest.cc b/components/sync/core/processor_entity_tracker_unittest.cc
index 58c10c81dde0c6058e49158fe5116c5da427bf41..aa48d3774626485b139673441511f313f56c2c07 100644
--- a/components/sync/core/processor_entity_tracker_unittest.cc
+++ b/components/sync/core/processor_entity_tracker_unittest.cc
@@ -469,6 +469,12 @@ TEST_F(ProcessorEntityTrackerTest, LocalChangesInterleaved) {
EXPECT_EQ(specifics_hash_v2, entity->metadata().specifics_hash());
EXPECT_EQ(specifics_hash_v1, entity->metadata().base_specifics_hash());
+ EXPECT_TRUE(entity->IsUnsynced());
+ EXPECT_FALSE(entity->RequiresCommitRequest());
+ EXPECT_FALSE(entity->RequiresCommitData());
+ EXPECT_FALSE(entity->CanClearMetadata());
+ EXPECT_TRUE(entity->HasCommitData());
+
// Ack the second commit.
entity->ReceiveCommitResponse(GenerateAckData(request_v2, kId, 3));
« no previous file with comments | « no previous file | components/sync/engine_impl/model_type_worker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698