Suggest to fill password change forms

components/autofill/content/renderer/password_form_conversion_utils.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/autofill/content/renderer/password_form_conversion_utils.h"
 
 #include <vector>
 
 #include "base/lazy_instance.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 125 matching lines @@
 
   if (passwords.empty())
     return false;
 
   switch (passwords.size()) {
     case 1:
       // Single password, easy.
       *current_password = passwords[0];
       break;
     case 2:
-      if (passwords[0].value() == passwords[1].value()) {
+      if (!passwords[0].value().isEmpty() &&
+          passwords[0].value() == passwords[1].value()) {

[Garrett Casto, 2015/06/05 21:22:30]

You probably want to leave a comment here about why you want to have two empty
password fields count as a change password form not as a signup form.

[dvadym, 2015/06/09 16:40:08]

Done.

         // Two identical passwords: assume we are seeing a new password with a
         // confirmation. This can be either a sign-up form or a password change
         // form that does not ask for the old password.
         *new_password = passwords[0];
       } else {
         // Assume first is old password, second is new (no choice but to guess).
         *current_password = passwords[0];
         *new_password = passwords[1];
       }
       break;
@@ skipping 325 matching lines @@
   WebFormElementToFormData(web_form,
                            blink::WebFormControlElement(),
                            EXTRACT_NONE,
                            &password_form->form_data,
                            NULL /* FormFieldData */);
 
   return password_form.Pass();
 }
 
 }  // namespace autofill