diff options
| author | maruel@chromium.org <maruel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-22 11:56:41 +0000 |
|---|---|---|
| committer | maruel@chromium.org <maruel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-22 11:56:41 +0000 |
| commit | a54a51fdcee61c0b04f7b9cc7ccad296cf3b23a9 (patch) | |
| tree | 9dfdb313cc2433a8406752103282add87a135f9e /sync/internal_api/sync_manager_impl.cc | |
| parent | 8b77800e991ef7880b9afeff81018fe82d57e2ca (diff) | |
| download | chromium_src-a54a51fdcee61c0b04f7b9cc7ccad296cf3b23a9.zip chromium_src-a54a51fdcee61c0b04f7b9cc7ccad296cf3b23a9.tar.gz chromium_src-a54a51fdcee61c0b04f7b9cc7ccad296cf3b23a9.tar.bz2 | |
Revert r147801 "Refactor sync-specific parts out of SyncNotifier/SyncNotifierObserver"
This broke sync_integration_tests like it had never been run.
Note that sync_integration_tests is not on GateKeeper but redness is still sad.
TBR=dcheng@chromium.org
BUG=
TEST=
Review URL: https://chromiumcodereview.appspot.com/10806060
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147817 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/internal_api/sync_manager_impl.cc')
| -rw-r--r-- | sync/internal_api/sync_manager_impl.cc | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/sync/internal_api/sync_manager_impl.cc b/sync/internal_api/sync_manager_impl.cc index 98ba4a3..547dae8 100644 --- a/sync/internal_api/sync_manager_impl.cc +++ b/sync/internal_api/sync_manager_impl.cc @@ -39,7 +39,6 @@ #include "sync/js/js_event_details.h" #include "sync/js/js_event_handler.h" #include "sync/js/js_reply_handler.h" -#include "sync/notifier/invalidation_util.h" #include "sync/notifier/notifications_disabled_reason.h" #include "sync/notifier/sync_notifier.h" #include "sync/protocol/encryption.pb.h" @@ -482,6 +481,8 @@ bool SyncManagerImpl::Init( if (!success) return false; + sync_notifier_->AddObserver(this); + return success; } @@ -724,8 +725,7 @@ void SyncManagerImpl::UpdateCredentials( void SyncManagerImpl::UpdateEnabledTypes( const ModelTypeSet& enabled_types) { DCHECK(thread_checker_.CalledOnValidThread()); - sync_notifier_->UpdateRegisteredIds(this, - ModelTypeSetToObjectIdSet(enabled_types)); + sync_notifier_->UpdateEnabledTypes(enabled_types); } void SyncManagerImpl::SetEncryptionPassphrase( @@ -1195,7 +1195,7 @@ void SyncManagerImpl::ShutdownOnSyncThread() { RemoveObserver(&debug_info_event_listener_); if (sync_notifier_.get()) { - sync_notifier_->UpdateRegisteredIds(this, ObjectIdSet()); + sync_notifier_->RemoveObserver(this); } sync_notifier_.reset(); @@ -1781,11 +1781,9 @@ void SyncManagerImpl::OnNotificationsDisabled( } void SyncManagerImpl::OnIncomingNotification( - const ObjectIdPayloadMap& id_payloads, + const ModelTypePayloadMap& type_payloads, IncomingNotificationSource source) { DCHECK(thread_checker_.CalledOnValidThread()); - const ModelTypePayloadMap& type_payloads = - ObjectIdPayloadMapToModelTypePayloadMap(id_payloads); if (source == LOCAL_NOTIFICATION) { scheduler_->ScheduleNudgeWithPayloadsAsync( TimeDelta::FromMilliseconds(kSyncRefreshDelayMsec), |