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

Unified Diff: sync/internal_api/sync_manager_impl.cc

Issue 10805002: [Sync] Enable adding notifier observers from ProfileSyncService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix typos 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/internal_api/sync_manager_impl.cc
diff --git a/sync/internal_api/sync_manager_impl.cc b/sync/internal_api/sync_manager_impl.cc
index 7e173d15fa2f7de34acda28501b7d6db394abba1..81579f676278e9f0295f78d57c9e4932d7b19ece 100644
--- a/sync/internal_api/sync_manager_impl.cc
+++ b/sync/internal_api/sync_manager_impl.cc
@@ -734,8 +734,15 @@ void SyncManagerImpl::UpdateCredentials(
void SyncManagerImpl::UpdateEnabledTypes(
const ModelTypeSet& enabled_types) {
DCHECK(thread_checker_.CalledOnValidThread());
- sync_notifier_->UpdateRegisteredIds(this,
- ModelTypeSetToObjectIdSet(enabled_types));
+ sync_notifier_->UpdateRegisteredIds(
+ this,
+ ModelTypeSetToObjectIdSet(enabled_types));
+}
+
+void SyncManagerImpl::UpdateRegisteredIds(
tim (not reviewing) 2012/08/01 21:52:52 It'd be nice, maybe just while it's still explicit
akalin 2012/08/02 08:05:43 Done. Renamed to UpdateRegisteredInvalidationIds.
+ SyncNotifierObserver* handler, const ObjectIdSet& ids) {
+ DCHECK(thread_checker_.CalledOnValidThread());
+ sync_notifier_->UpdateRegisteredIds(handler, ids);
}
void SyncManagerImpl::SetEncryptionPassphrase(

Powered by Google App Engine
This is Rietveld 408576698