diff options
| author | dcheng@chromium.org <dcheng@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-27 02:02:00 +0000 |
|---|---|---|
| committer | dcheng@chromium.org <dcheng@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-27 02:02:00 +0000 |
| commit | d914f0238aa455cdb25f0782839555ec20d4be02 (patch) | |
| tree | 4f60c5d8cc098eb34aa90ff015bd121e1668f4dc /sync/notifier/sync_notifier_factory_unittest.cc | |
| parent | df52d2b635484f73cad2357e2e422d0bcf1de68f (diff) | |
| download | chromium_src-d914f0238aa455cdb25f0782839555ec20d4be02.zip chromium_src-d914f0238aa455cdb25f0782839555ec20d4be02.tar.gz chromium_src-d914f0238aa455cdb25f0782839555ec20d4be02.tar.bz2 | |
Refactor sync-specific parts out of SyncNotifier/SyncNotifierObserver
Sort of. SendNotification() is still there. Perhaps we want to split the interfaces completely.
BUG=124149
TEST=tests should still pass, no observable behavior change
NOTRY=true
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=147801
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148496
Review URL: https://chromiumcodereview.appspot.com/10702074
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148697 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/notifier/sync_notifier_factory_unittest.cc')
| -rw-r--r-- | sync/notifier/sync_notifier_factory_unittest.cc | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/sync/notifier/sync_notifier_factory_unittest.cc b/sync/notifier/sync_notifier_factory_unittest.cc index 52fd505..8036c1a 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 } |