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

Unified Diff: chrome/browser/sync/test/integration/sync_test.cc

Issue 10874096: [Sync] Rework Invalidator-related unit tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 8 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 | « chrome/browser/sync/glue/sync_backend_host.cc ('k') | sync/internal_api/public/base/invalidation_state.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/test/integration/sync_test.cc
diff --git a/chrome/browser/sync/test/integration/sync_test.cc b/chrome/browser/sync/test/integration/sync_test.cc
index 9fb7eb4933e048d2091a8c49f6fc96936aaa4013..bc28f73a843319133bae80d601d78750ceddf955 100644
--- a/chrome/browser/sync/test/integration/sync_test.cc
+++ b/chrome/browser/sync/test/integration/sync_test.cc
@@ -651,9 +651,12 @@ void SyncTest::EnableNotifications() {
void SyncTest::TriggerNotification(syncer::ModelTypeSet changed_types) {
ASSERT_TRUE(ServerSupportsNotificationControl());
const std::string& data =
- syncer::P2PNotificationData("from_server",
- syncer::NOTIFY_ALL,
- changed_types).ToString();
+ syncer::P2PNotificationData(
+ "from_server",
+ syncer::NOTIFY_ALL,
+ syncer::ObjectIdSetToStateMap(
+ syncer::ModelTypeSetToObjectIdSet(changed_types), ""),
+ syncer::REMOTE_NOTIFICATION).ToString();
const std::string& path =
std::string("chromiumsync/sendnotification?channel=") +
syncer::kSyncP2PNotificationChannel + "&data=" + data;
« no previous file with comments | « chrome/browser/sync/glue/sync_backend_host.cc ('k') | sync/internal_api/public/base/invalidation_state.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698