Add username field discovery heuristic.

components/autofill/content/renderer/password_autofill_agent.h

 // 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.
 
 #ifndef COMPONENTS_AUTOFILL_CONTENT_RENDERER_PASSWORD_AUTOFILL_AGENT_H_
 #define COMPONENTS_AUTOFILL_CONTENT_RENDERER_PASSWORD_AUTOFILL_AGENT_H_
 
 #include <map>
 #include <memory>
 #include <vector>
@@ skipping 211 matching lines @@
   // be filled. |show_in_password_field| should indicate whether the pop-up is
   // to be shown on the password field instead of on the username field. If the
   // username exists, it should be passed as |user_input|. If there is no
   // username, pass the password field in |user_input|. In the latter case, no
   // username value will be shown in the pop-up.
   bool ShowSuggestionPopup(const PasswordInfo& password_info,
                            const blink::WebInputElement& user_input,
                            bool show_all,
                            bool show_on_password_field);
 
-  // Finds the PasswordInfo, username and password fields that corresponds to
-  // the passed in |element|. |element| can refer either to a username element
-  // or a password element. If a PasswordInfo was found, returns |true| and also
-  // assigns the corresponding username, password elements and PasswordInfo into
+  // Finds the PasswordInfo, username and password fields corresponding to the
+  // passed in |element|, which can refer to either a username or a password
+  // element. If a PasswordInfo was found, returns |true| and assigns the
+  // corresponding username, password elements and PasswordInfo into
   // |username_element|, |password_element| and |pasword_info|, respectively.
-  // Note, that |username_element->isNull()| can be true if |element| is a
+  // Note, that |username_element->IsNull()| can be true if |element| is a
   // password.
   bool FindPasswordInfoForElement(const blink::WebInputElement& element,
                                   blink::WebInputElement* username_element,
                                   blink::WebInputElement* password_element,
                                   PasswordInfo** password_info);
 
   // Invoked when the frame is closing.
   void FrameClosing();
 
   // Clears the preview for the username and password fields, restoring both to
@@ skipping 13 matching lines @@
                                  ProvisionallySaveRestriction restriction);
 
   // Helper function called when same-document navigation completed
   void OnSameDocumentNavigationCompleted(
       PasswordForm::SubmissionIndicatorEvent event);
 
   const mojom::AutofillDriverPtr& GetAutofillDriver();
 
   // The logins we have filled so far with their associated info.
   WebInputToPasswordInfoMap web_input_to_password_info_;
-  // A (sort-of) reverse map to |login_to_password_info_|.
+  // A (sort-of) reverse map to |web_input_to_password_info_|.
   PasswordToLoginMap password_to_username_;
 
   // Set if the user might be submitting a password form on the current page,
   // but the submit may still fail (i.e. doesn't pass JavaScript validation).
   ProvisionallySavedPasswordForm provisionally_saved_form_;
 
   // Map WebFormControlElement to the pair of:
   // 1) The most recent text that user typed or PasswordManager autofilled in
   // input elements. Used for storing username/password before JavaScript
   // changes them.
@@ skipping 31 matching lines @@
   mojo::Binding<mojom::PasswordAutofillAgent> binding_;
 
   blink::WebFormElementObserver* form_element_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(PasswordAutofillAgent);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CONTENT_RENDERER_PASSWORD_AUTOFILL_AGENT_H_