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

Unified Diff: chrome/browser/history/history_unittest.cc

Issue 217063005: Separate SyncData methods into three groups, local, remote, and common. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@syncapi
Patch Set: Created 6 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
Index: chrome/browser/history/history_unittest.cc
diff --git a/chrome/browser/history/history_unittest.cc b/chrome/browser/history/history_unittest.cc
index 803748ec699ecbbad11da68ad3d10d213de6c5cd..21158ec2b625f55e7ec1647631afe2241b99486b 100644
--- a/chrome/browser/history/history_unittest.cc
+++ b/chrome/browser/history/history_unittest.cc
@@ -1683,9 +1683,9 @@ TEST_F(HistoryTest, ProcessGlobalIdDeleteDirective) {
const syncer::SyncChangeList& sync_changes = change_processor.changes();
ASSERT_EQ(2u, sync_changes.size());
EXPECT_EQ(syncer::SyncChange::ACTION_DELETE, sync_changes[0].change_type());
- EXPECT_EQ(1, sync_changes[0].sync_data().GetRemoteId());
+ EXPECT_EQ(1, sync_changes[0].sync_data().remote().GetRemoteId());
EXPECT_EQ(syncer::SyncChange::ACTION_DELETE, sync_changes[1].change_type());
- EXPECT_EQ(2, sync_changes[1].sync_data().GetRemoteId());
+ EXPECT_EQ(2, sync_changes[1].sync_data().remote().GetRemoteId());
}
// Create delete directives for time ranges. The expected entries should be
@@ -1758,9 +1758,9 @@ TEST_F(HistoryTest, ProcessTimeRangeDeleteDirective) {
const syncer::SyncChangeList& sync_changes = change_processor.changes();
ASSERT_EQ(2u, sync_changes.size());
EXPECT_EQ(syncer::SyncChange::ACTION_DELETE, sync_changes[0].change_type());
- EXPECT_EQ(1, sync_changes[0].sync_data().GetRemoteId());
+ EXPECT_EQ(1, sync_changes[0].sync_data().remote().GetRemoteId());
EXPECT_EQ(syncer::SyncChange::ACTION_DELETE, sync_changes[1].change_type());
- EXPECT_EQ(2, sync_changes[1].sync_data().GetRemoteId());
+ EXPECT_EQ(2, sync_changes[1].sync_data().remote().GetRemoteId());
}
TEST_F(HistoryBackendDBTest, MigratePresentations) {

Powered by Google App Engine
This is Rietveld 408576698