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

chrome/browser/sync/glue/password_model_worker.h

Index: chrome/browser/sync/glue/password_model_worker.h
diff --git a/chrome/browser/sync/glue/password_model_worker.h b/chrome/browser/sync/glue/password_model_worker.h
index 5fe13268a1be14f5da219bfdb9cfc8bbdf170718..ac4030e086eebf579362a16a4b5de9e7759845be 100644
--- a/chrome/browser/sync/glue/password_model_worker.h
+++ b/chrome/browser/sync/glue/password_model_worker.h
@@ -31,6 +31,7 @@ class PasswordModelWorker : public syncer::ModelSafeWorker {
   // syncer::ModelSafeWorker implementation. Called on syncapi SyncerThread.
   virtual void RegisterForLoopDestruction() OVERRIDE;
   virtual syncer::ModelSafeGroup GetModelSafeGroup() OVERRIDE;
+  virtual void RequestStop() OVERRIDE;
 
  protected:
   virtual syncer::SyncerError DoWorkAndWaitUntilDoneImpl(
@@ -48,6 +49,7 @@ class PasswordModelWorker : public syncer::ModelSafeWorker {
   // observer.
   void RegisterForPasswordLoopDestruction();
 
+  base::Lock password_store_lock_;

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

Comment why a lock is needed to protect the password store (i.e. it's used on
password thread but released on UI thread).

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

Done.

   scoped_refptr<PasswordStore> password_store_;
   DISALLOW_COPY_AND_ASSIGN(PasswordModelWorker);
 };