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

Unified Diff: chrome/browser/sync/glue/chrome_sync_notification_bridge.h

Issue 10824161: [Sync] Avoid unregistering object IDs on shutdown (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove now-unneeded param 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/chrome_sync_notification_bridge.h
diff --git a/chrome/browser/sync/glue/chrome_sync_notification_bridge.h b/chrome/browser/sync/glue/chrome_sync_notification_bridge.h
index a878ff711f14e999c7282a637f1108433ef7d8b0..64ea85c71ba7ee3a391a94da953f2150faa2b648 100644
--- a/chrome/browser/sync/glue/chrome_sync_notification_bridge.h
+++ b/chrome/browser/sync/glue/chrome_sync_notification_bridge.h
@@ -36,10 +36,17 @@ class ChromeSyncNotificationBridge : public content::NotificationObserver {
const scoped_refptr<base::SequencedTaskRunner>& sync_task_runner);
virtual ~ChromeSyncNotificationBridge();
+ // Must be called on the UI thread while the sync task runner is
+ // still around. No other member functions on the sync thread may
+ // be called after this is called.
+ void StopForShutdown();
+
// Must be called on the sync task runner.
void UpdateEnabledTypes(syncer::ModelTypeSet enabled_types);
// Marked virtual for tests.
- virtual void UpdateRegisteredIds(syncer::SyncNotifierObserver* handler,
+ virtual void SetHandler(const std::string& handler_name,
+ syncer::SyncNotifierObserver* handler);
+ virtual void UpdateRegisteredIds(const std::string& handler_name,
const syncer::ObjectIdSet& ids);
// NotificationObserver implementation. Called on UI thread.

Powered by Google App Engine
This is Rietveld 408576698