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

Unified Diff: sync/notifier/sync_notifier_factory_unittest.cc

Issue 10702074: Refactor sync-specific parts out of SyncNotifier/SyncNotifierObserver (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: DEPS Created 8 years, 5 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: sync/notifier/sync_notifier_factory_unittest.cc
diff --git a/sync/notifier/sync_notifier_factory_unittest.cc b/sync/notifier/sync_notifier_factory_unittest.cc
index e11d9cfaa0453e4ffd522423e5ecdb25eb45eee7..5cf65b3513ceea4d50f1759af4f8a238f68425e0 100644
--- a/sync/notifier/sync_notifier_factory_unittest.cc
+++ b/sync/notifier/sync_notifier_factory_unittest.cc
@@ -60,8 +60,9 @@ TEST_F(SyncNotifierFactoryTest, Basic) {
ASSERT_FALSE(notifier.get());
#else
ASSERT_TRUE(notifier.get());
- notifier->AddObserver(&mock_observer_);
- notifier->RemoveObserver(&mock_observer_);
+ ObjectIdSet ids = ModelTypeSetToObjectIdSet(ModelTypeSet(syncer::BOOKMARKS));
+ notifier->UpdateRegisteredIds(&mock_observer_, ids);
+ notifier->UpdateRegisteredIds(&mock_observer_, ObjectIdSet());
#endif
}
@@ -77,8 +78,9 @@ TEST_F(SyncNotifierFactoryTest, Basic_P2P) {
ASSERT_FALSE(notifier.get());
#else
ASSERT_TRUE(notifier.get());
- notifier->AddObserver(&mock_observer_);
- notifier->RemoveObserver(&mock_observer_);
+ ObjectIdSet ids = ModelTypeSetToObjectIdSet(ModelTypeSet(syncer::BOOKMARKS));
+ notifier->UpdateRegisteredIds(&mock_observer_, ids);
+ notifier->UpdateRegisteredIds(&mock_observer_, ObjectIdSet());
#endif
}

Powered by Google App Engine
This is Rietveld 408576698