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

Unified Diff: chrome/browser/sync/glue/bridged_sync_notifier.cc

Issue 10823037: Revert r148496 "Refactor sync-specific parts out of SyncNotifier/SyncNotifierObserver" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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: chrome/browser/sync/glue/bridged_sync_notifier.cc
diff --git a/chrome/browser/sync/glue/bridged_sync_notifier.cc b/chrome/browser/sync/glue/bridged_sync_notifier.cc
index 5e3f669937da088a6d12e30bb9938886a34af1fe..a9de0577de66a24b7cdadec89b28bd87952e21db 100644
--- a/chrome/browser/sync/glue/bridged_sync_notifier.cc
+++ b/chrome/browser/sync/glue/bridged_sync_notifier.cc
@@ -18,12 +18,18 @@ BridgedSyncNotifier::BridgedSyncNotifier(
BridgedSyncNotifier::~BridgedSyncNotifier() {
}
-void BridgedSyncNotifier::UpdateRegisteredIds(
- syncer::SyncNotifierObserver* handler,
- const syncer::ObjectIdSet& ids) {
+void BridgedSyncNotifier::AddObserver(
+ syncer::SyncNotifierObserver* observer) {
if (delegate_.get())
- delegate_->UpdateRegisteredIds(handler, ids);
- bridge_->UpdateRegisteredIds(handler, ids);
+ delegate_->AddObserver(observer);
+ bridge_->AddObserver(observer);
+}
+
+void BridgedSyncNotifier::RemoveObserver(
+ syncer::SyncNotifierObserver* observer) {
+ bridge_->RemoveObserver(observer);
+ if (delegate_.get())
+ delegate_->RemoveObserver(observer);
}
void BridgedSyncNotifier::SetUniqueId(const std::string& unique_id) {
@@ -42,6 +48,12 @@ void BridgedSyncNotifier::UpdateCredentials(
delegate_->UpdateCredentials(email, token);
}
+void BridgedSyncNotifier::UpdateEnabledTypes(
+ syncer::ModelTypeSet enabled_types) {
+ if (delegate_.get())
+ delegate_->UpdateEnabledTypes(enabled_types);
+}
+
void BridgedSyncNotifier::SendNotification(
syncer::ModelTypeSet changed_types) {
if (delegate_.get())
« no previous file with comments | « chrome/browser/sync/glue/bridged_sync_notifier.h ('k') | chrome/browser/sync/glue/bridged_sync_notifier_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698