Index: chrome/browser/password_manager/password_store_x.cc |
=================================================================== |
--- chrome/browser/password_manager/password_store_x.cc (revision 51079) |
+++ chrome/browser/password_manager/password_store_x.cc (working copy) |
@@ -220,9 +220,13 @@ |
// where some fail. The only real problem with this is that we might |
// leave passwords in the login database and never come back to clean |
// them out if any of these calls do fail. |
- // TODO(mdm): Really we should just delete the login database file. |
PasswordStoreDefault::RemoveLoginImpl(*forms[i]); |
} |
+ // Finally, delete the database file itself. We remove the passwords from |
+ // it before deleting the file just in case there is some problem deleting |
+ // the file (e.g. directory is not writable, but file is), which would |
+ // otherwise cause passwords to re-migrate next (or maybe every) time. |
+ DeleteAndRecreateDatabaseFile(); |
} |
} |
ssize_t result = ok ? forms.size() : -1; |