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

Unified Diff: chrome/browser/password_manager/password_store_factory.cc

Issue 10227014: Remove the very old code that migrated password data from the WebDataService to PasswordStore. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 8 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_factory.cc
===================================================================
--- chrome/browser/password_manager/password_store_factory.cc (revision 133948)
+++ chrome/browser/password_manager/password_store_factory.cc (working copy)
@@ -118,8 +118,7 @@
#elif defined(OS_CHROMEOS) || defined(OS_ANDROID)
// For now, we use PasswordStoreDefault. We might want to make a native
// backend for PasswordStoreX (see below) in the future though.
- ps = new PasswordStoreDefault(
- login_db, profile, profile->GetWebDataService(Profile::IMPLICIT_ACCESS));
+ ps = new PasswordStoreDefault(login_db, profile);
#elif defined(OS_POSIX)
// On POSIX systems, we try to use the "native" password management system of
// the desktop environment currently running, allowing GNOME Keyring in XFCE.
@@ -173,9 +172,7 @@
"more information about password storage options.";
}
- ps = new PasswordStoreX(login_db, profile,
- profile->GetWebDataService(Profile::IMPLICIT_ACCESS),
- backend.release());
+ ps = new PasswordStoreX(login_db, profile, backend.release());
#else
NOTIMPLEMENTED();
#endif

Powered by Google App Engine
This is Rietveld 408576698