Release password store on UI loop when worker is stopped so that password

sync/internal_api/public/engine/model_safe_worker.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef SYNC_INTERNAL_API_PUBLIC_ENGINE_MODEL_SAFE_WORKER_H_
 #define SYNC_INTERNAL_API_PUBLIC_ENGINE_MODEL_SAFE_WORKER_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 66 matching lines @@
   // |unregister_done_callback|.
   virtual void UnregisterForLoopDestruction(
       base::Callback<void(ModelSafeGroup)> unregister_done_callback);
 
   // If not stopped, call DoWorkAndWaitUntilDoneImpl() to do work. Otherwise
   // return CANNOT_DO_WORK.
   SyncerError DoWorkAndWaitUntilDone(const WorkCallback& work);
 
   // Soft stop worker by setting stopped_ flag. Called when sync is disabled
   // or browser is shutting down.
-  void RequestStop();
+  virtual void RequestStop();

[Nicolas Zea, 2013/09/16 17:37:51]

Is this always called on the UI thread? Perhaps add a comment to that effect?

[haitaol1, 2013/09/16 18:07:39]

Done.

 
   virtual ModelSafeGroup GetModelSafeGroup() = 0;
 
   // MessageLoop::DestructionObserver implementation.
   virtual void WillDestroyCurrentMessageLoop() OVERRIDE;
 
  protected:
   friend class base::RefCountedThreadSafe<ModelSafeWorker>;
 
   explicit ModelSafeWorker(WorkerLoopDestructionObserver* observer);
@@ skipping 53 matching lines @@
 SYNC_EXPORT ModelTypeSet GetRoutingInfoTypes(
     const ModelSafeRoutingInfo& routing_info);
 
 SYNC_EXPORT ModelSafeGroup GetGroupForModelType(
     const ModelType type,
     const ModelSafeRoutingInfo& routes);
 
 }  // namespace syncer
 
 #endif  // SYNC_INTERNAL_API_PUBLIC_ENGINE_MODEL_SAFE_WORKER_H_