Fix DCHECK, memory leak, and refactor PasswordStore to use CancelableRequest

chrome/browser/password_manager/password_store_win.h

Index: chrome/browser/password_manager/password_store_win.h
diff --git a/chrome/browser/password_manager/password_store_win.h b/chrome/browser/password_manager/password_store_win.h
index e58f71723b781cb7ffb9324320b5a7de8e051b48..d3cc61dfb15fc921c10476d91582da3ecd019385 100644
--- a/chrome/browser/password_manager/password_store_win.h
+++ b/chrome/browser/password_manager/password_store_win.h
@@ -34,9 +34,7 @@ class PasswordStoreWin : public PasswordStoreDefault {
   void OnWebDataServiceRequestDone(WebDataService::Handle h,
                                    const WDTypedResult* result);
 
-  virtual void NotifyConsumer(
-      GetLoginsRequest* request,
-      const std::vector<webkit_glue::PasswordForm*> forms);
+  virtual void ForwardLoginsResult(GetLoginsRequest* request);
 
   // Takes ownership of |request| and tracks it under |handle|.
   void TrackRequest(WebDataService::Handle handle, GetLoginsRequest* request);