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

Unified Diff: chrome/browser/password_manager/password_store.h

Issue 9665007: Profile refactoring: Remove all PasswordStore code from the Profile interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add to factory list Created 8 years, 9 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/password_manager/password_store.h
diff --git a/chrome/browser/password_manager/password_store.h b/chrome/browser/password_manager/password_store.h
index 6613dd10cad89ec9908344a012877113d6327253..dda59d8e7b4ac804c5cba6864221fd4e13dced7f 100644
--- a/chrome/browser/password_manager/password_store.h
+++ b/chrome/browser/password_manager/password_store.h
@@ -14,6 +14,7 @@
#include "base/threading/thread.h"
#include "base/time.h"
#include "chrome/browser/cancelable_request.h"
+#include "chrome/browser/profiles/refcounted_profile_keyed_service.h"
class PasswordStore;
class PasswordStoreConsumer;
@@ -41,7 +42,7 @@ void UpdateLogin(PasswordStore* store, const webkit::forms::PasswordForm& form);
// The login request/manipulation API is not threadsafe and must be used
// from the UI thread.
class PasswordStore
- : public base::RefCountedThreadSafe<PasswordStore>,
+ : public RefcountedProfileKeyedService,
public CancelableRequestProvider {
public:
typedef base::Callback<
@@ -84,9 +85,6 @@ class PasswordStore
// Reimplement this to add custom initialization. Always call this too.
virtual bool Init();
- // Invoked from the profiles destructor to shutdown the PasswordStore.
- virtual void Shutdown();
-
// Adds the given PasswordForm to the secure password store asynchronously.
virtual void AddLogin(const webkit::forms::PasswordForm& form);
« no previous file with comments | « chrome/browser/password_manager/password_manager_unittest.cc ('k') | chrome/browser/password_manager/password_store.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698