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

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

Issue 2222373003: [Sync] Adding storage key concept for ModelTypeServices. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removing redundant hash value. Created 4 years, 4 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 | « components/sync/core/data_batch_impl.cc ('k') | components/sync/core/processor_entity_tracker.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/core/data_batch_impl_unittest.cc
diff --git a/components/sync/core/data_batch_impl_unittest.cc b/components/sync/core/data_batch_impl_unittest.cc
index 3a51a4912ea8a5934e11463b467a17fb2c7b571d..86025cb6363153037dce745a5296edcff70411aa 100644
--- a/components/sync/core/data_batch_impl_unittest.cc
+++ b/components/sync/core/data_batch_impl_unittest.cc
@@ -19,7 +19,7 @@ TEST(DataBatchImplTest, PutAndNextWithReuse) {
batch.Put("one", base::WrapUnique(entity1));
EXPECT_TRUE(batch.HasNext());
- const TagAndData& pair1 = batch.Next();
+ const KeyAndData& pair1 = batch.Next();
EXPECT_FALSE(batch.HasNext());
EXPECT_EQ("one", pair1.first);
EXPECT_EQ(entity1, pair1.second.get());
@@ -27,7 +27,7 @@ TEST(DataBatchImplTest, PutAndNextWithReuse) {
batch.Put("two", base::WrapUnique(entity2));
EXPECT_TRUE(batch.HasNext());
- const TagAndData& pair2 = batch.Next();
+ const KeyAndData& pair2 = batch.Next();
EXPECT_FALSE(batch.HasNext());
EXPECT_EQ("two", pair2.first);
EXPECT_EQ(entity2, pair2.second.get());
@@ -46,7 +46,7 @@ TEST(DataBatchImplTest, PutAndNextInterleaved) {
batch.Put("two", base::WrapUnique(entity2));
EXPECT_TRUE(batch.HasNext());
- const TagAndData& pair1 = batch.Next();
+ const KeyAndData& pair1 = batch.Next();
EXPECT_TRUE(batch.HasNext());
EXPECT_EQ("one", pair1.first);
EXPECT_EQ(entity1, pair1.second.get());
@@ -54,18 +54,18 @@ TEST(DataBatchImplTest, PutAndNextInterleaved) {
batch.Put("three", base::WrapUnique(entity3));
EXPECT_TRUE(batch.HasNext());
- const TagAndData& pair2 = batch.Next();
+ const KeyAndData& pair2 = batch.Next();
EXPECT_TRUE(batch.HasNext());
EXPECT_EQ("two", pair2.first);
EXPECT_EQ(entity2, pair2.second.get());
- const TagAndData& pair3 = batch.Next();
+ const KeyAndData& pair3 = batch.Next();
EXPECT_FALSE(batch.HasNext());
EXPECT_EQ("three", pair3.first);
EXPECT_EQ(entity3, pair3.second.get());
}
-TEST(DataBatchImplTest, PutAndNextSharedTag) {
+TEST(DataBatchImplTest, PutAndNextSharedKey) {
EntityData* entity1 = new EntityData();
EntityData* entity2 = new EntityData();
@@ -77,12 +77,12 @@ TEST(DataBatchImplTest, PutAndNextSharedTag) {
batch.Put("same", base::WrapUnique(entity2));
EXPECT_TRUE(batch.HasNext());
- const TagAndData& pair1 = batch.Next();
+ const KeyAndData& pair1 = batch.Next();
EXPECT_TRUE(batch.HasNext());
EXPECT_EQ("same", pair1.first);
EXPECT_EQ(entity1, pair1.second.get());
- const TagAndData& pair2 = batch.Next();
+ const KeyAndData& pair2 = batch.Next();
EXPECT_FALSE(batch.HasNext());
EXPECT_EQ("same", pair2.first);
EXPECT_EQ(entity2, pair2.second.get());
« no previous file with comments | « components/sync/core/data_batch_impl.cc ('k') | components/sync/core/processor_entity_tracker.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698