Worker changes to prepare for lock-free shutdown.

chrome/browser/sync/glue/sync_backend_registrar.h

Index: chrome/browser/sync/glue/sync_backend_registrar.h
diff --git a/chrome/browser/sync/glue/sync_backend_registrar.h b/chrome/browser/sync/glue/sync_backend_registrar.h
index f43e2791318838fe54e9e2db9269c73886c065ab..9bed0bb7fb64f3cdd1cb1ae25d07aa5e002d2d90 100644
--- a/chrome/browser/sync/glue/sync_backend_registrar.h
+++ b/chrome/browser/sync/glue/sync_backend_registrar.h
@@ -34,7 +34,8 @@ class UIModelWorker;
 // A class that keep track of the workers, change processors, and
 // routing info for the enabled sync types, and also routes change
 // events to the right processors.
-class SyncBackendRegistrar : public syncer::SyncManager::ChangeDelegate {
+class SyncBackendRegistrar : public syncer::SyncManager::ChangeDelegate,
+                             public syncer::WorkerObserver {

[tim (not reviewing), 2013/05/14 04:14:29]

WorkerDestructionObserver would be a better name for this, particularly since it
is just a relay of MessageLoop::DestructionObserver.

[haitaol1, 2013/05/15 23:39:21]

I think WorkerLoopDesctructionObserver is more accurate.

On 2013/05/14 04:14:29, timsteele wrote:

  public:
   // |name| is used for debugging.  Does not take ownership of |profile| or
   // |sync_loop|.  Must be created on the UI thread.
@@ -108,6 +109,9 @@ class SyncBackendRegistrar : public syncer::SyncManager::ChangeDelegate {
   void GetWorkers(std::vector<syncer::ModelSafeWorker*>* out);
   void GetModelSafeRoutingInfo(syncer::ModelSafeRoutingInfo* out);
 
+  // syncer::WorkerObserver implementation.
+  virtual void OnWorkerDisabled(syncer::ModelSafeGroup group) OVERRIDE;
+
  private:
   typedef std::map<syncer::ModelSafeGroup,
                    scoped_refptr<syncer::ModelSafeWorker> > WorkerMap;