Ignore autocomplete=off when password manager is saving the generated passwords during account crea…

chrome/browser/password_manager/password_manager.cc

Index: chrome/browser/password_manager/password_manager.cc
diff --git a/chrome/browser/password_manager/password_manager.cc b/chrome/browser/password_manager/password_manager.cc
index a00c991d1ccba8542a4bff4bd0f61435a23cfbbb..6990d52567dbe438428cb77270489ab43dce9efd 100644
--- a/chrome/browser/password_manager/password_manager.cc
+++ b/chrome/browser/password_manager/password_manager.cc
@@ -181,6 +181,12 @@ void PasswordManager::ProvisionallySavePassword(const PasswordForm& form) {
   if (!manager->HasValidPasswordForm())
     return;
 
+  // Don't save the password if it is not generated password and autocomplete is

[Garrett Casto, 2012/12/06 08:06:14]

This sentence doesn't read right. I think you want to either say "not a
generated password" or "not automatically generated". I also don't think that
you need the "for it".

+  // turned off for it.
+  if (!manager->HasGeneratedPassword() && !form.password_should_autocomplete) {

[Garrett Casto, 2012/12/06 08:06:14]

No braces here.

+    return;
+  }
+
   PasswordForm provisionally_saved_form(form);
   provisionally_saved_form.ssl_valid = form.origin.SchemeIsSecure() &&
       !delegate_->DidLastPageLoadEncounterSSLErrors();