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

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

Issue 10824161: [Sync] Avoid unregistering object IDs on shutdown (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Work around brittle unit test Created 8 years, 4 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..f7834188171a3a029dfcf4507babef66a70d580c 100644
--- a/chrome/browser/sync/glue/bridged_sync_notifier.cc
+++ b/chrome/browser/sync/glue/bridged_sync_notifier.cc
@@ -18,14 +18,28 @@ BridgedSyncNotifier::BridgedSyncNotifier(
BridgedSyncNotifier::~BridgedSyncNotifier() {
}
+void BridgedSyncNotifier::RegisterHandler(
+ syncer::SyncNotifierObserver* handler) {
+ if (delegate_.get())
+ delegate_->RegisterHandler(handler);
+ bridge_->RegisterHandler(handler);
+}
+
void BridgedSyncNotifier::UpdateRegisteredIds(
syncer::SyncNotifierObserver* handler,
const syncer::ObjectIdSet& ids) {
if (delegate_.get())
- delegate_->UpdateRegisteredIds(handler, ids);
+ delegate_->UpdateRegisteredIds(handler, ids);
bridge_->UpdateRegisteredIds(handler, ids);
}
+void BridgedSyncNotifier::UnregisterHandler(
+ syncer::SyncNotifierObserver* handler) {
+ if (delegate_.get())
+ delegate_->UnregisterHandler(handler);
+ bridge_->UnregisterHandler(handler);
+}
+
void BridgedSyncNotifier::SetUniqueId(const std::string& unique_id) {
if (delegate_.get())
delegate_->SetUniqueId(unique_id);
« 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