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

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

Issue 114057: Re-land the password store work from bug 8205, with changes that should fix b... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 7 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
« no previous file with comments | « chrome/browser/browser.vcproj ('k') | chrome/browser/password_manager/password_form_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/password_manager/password_form_manager.h
===================================================================
--- chrome/browser/password_manager/password_form_manager.h (revision 17245)
+++ chrome/browser/password_manager/password_form_manager.h (working copy)
@@ -8,6 +8,7 @@
#include "build/build_config.h"
#include "base/stl_util-inl.h"
+#include "chrome/browser/password_manager/password_store.h"
#include "chrome/browser/webdata/web_data_service.h"
#include "webkit/glue/password_form.h"
@@ -16,7 +17,7 @@
// Per-password-form-{on-page, dialog} class responsible for interactions
// between a given form, the per-tab PasswordManager, and the web database.
-class PasswordFormManager : public WebDataServiceConsumer {
+class PasswordFormManager : public PasswordStoreConsumer {
public:
// web_data_service allows access to current profile's Web Data
// password_manager owns this object
@@ -34,9 +35,6 @@
// Retrieves potential matching logins from the database.
void FetchMatchingLoginsFromWebDatabase();
-#if defined(OS_WIN)
- void FetchMatchingIE7LoginFromWebDatabase();
-#endif
// Simple state-check to verify whether this object as received a callback
// from the web database and completed its matching phase. Note that the
@@ -60,19 +58,12 @@
// managed by this.
bool IsNewLogin();
- // WebDataServiceConsumer implementation. If matches were found
- // (in *result), this is where we determine we need to autofill.
- virtual void OnWebDataServiceRequestDone(WebDataService::Handle h,
- const WDTypedResult* result);
-
// Determines if we need to autofill given the results of the query.
- void OnRequestDone(WebDataService::Handle h, const WDTypedResult* result);
+ void OnRequestDone(int handle, const std::vector<PasswordForm*>& result);
-#if defined(OS_WIN)
- // Determines if we need to autofill given the results of the query in the
- // ie7_password table.
- void OnIE7RequestDone(WebDataService::Handle h, const WDTypedResult* result);
-#endif
+ // PasswordStoreConsumer implementation.
+ virtual void OnPasswordStoreRequestDone(
+ int handle, const std::vector<PasswordForm*>& result);
// A user opted to 'never remember' passwords for this form.
// Blacklist it so that from now on when it is seen we ignore it.
@@ -141,7 +132,7 @@
const PasswordManager* const password_manager_;
// Handle to any pending WebDataService::GetLogins query.
- WebDataService::Handle pending_login_query_;
+ int pending_login_query_;
// Convenience pointer to entry in best_matches_ that is marked
// as preferred. This is only allowed to be null if there are no best matches
« no previous file with comments | « chrome/browser/browser.vcproj ('k') | chrome/browser/password_manager/password_form_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698