Add a basic functionality to the force-saving menu item.

components/autofill/content/renderer/password_autofill_agent.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_autofill_agent.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 949 matching lines @@
   }
 }
 
 bool PasswordAutofillAgent::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(PasswordAutofillAgent, message)
     IPC_MESSAGE_HANDLER(AutofillMsg_FillPasswordForm, OnFillPasswordForm)
     IPC_MESSAGE_HANDLER(AutofillMsg_SetLoggingState, OnSetLoggingState)
     IPC_MESSAGE_HANDLER(AutofillMsg_AutofillUsernameDataReceived,
                         OnAutofillUsernameDataReceived)
+    IPC_MESSAGE_HANDLER(AutofillMsg_ForceSavePassword, OnForceSavePassword)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 void PasswordAutofillAgent::DidFinishDocumentLoad() {
   // The |frame| contents have been parsed, but not yet rendered.  Let the
   // PasswordManager know that forms are loaded, even though we can't yet tell
   // whether they're visible.
   SendPasswordForms(false);
@@ skipping 254 matching lines @@
 
 void PasswordAutofillAgent::OnSetLoggingState(bool active) {
   logging_state_active_ = active;
 }
 
 void PasswordAutofillAgent::OnAutofillUsernameDataReceived(
     const FormDataFieldDataMap& predictions) {
   form_predictions_ = predictions;
 }
 
+void PasswordAutofillAgent::OnForceSavePassword() {
+  blink::WebElement element = render_frame()->GetFocusedElement();
+
+  DCHECK(!element.isNull());
+  DCHECK(element.hasHTMLTagName("input"));

[vabr (Chromium), 2015/05/28 19:41:23]

optional: Consider adding
<< element.tagName()
to the DCHECK for more information in case of failure.

[msramek, 2015/05/28 20:28:28]

Done.

+
+  blink::WebInputElement input = element.to<blink::WebInputElement>();
+  DCHECK(input.isPasswordField());
+
+  ProvisionallySavePassword(input.form(), RESTRICTION_NONE);

[vabr (Chromium), 2015/05/28 19:41:23]

Why do we need to provisionally save the password, when we send the credentials
to the browser on the next line?

[msramek, 2015/05/28 20:28:28]

Here, ProvisionallySavePassword() is used to transform WebFormElement into
PasswordForm, also applying autofill hints that this class got in
OnAutofillUsernameDataReceived.

The fact that the result is stored in provisionally_saved_form_ doesn't seem to
have any side-effects, especially because PasswordManager is indeed going to
offer to save that form after the callback from this method.

However, now that we changed the name of the method so that it doesn't refer to
force-saving, it will be of course cleaner if we just call CreatePasswordForm
instead.

[vabr (Chromium), 2015/05/29 06:48:50]

Acknowledged.

+  Send(new AutofillHostMsg_PasswordFormForceSaveRequested(
+      routing_id(), *provisionally_saved_form_));
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // PasswordAutofillAgent, private:
 
 PasswordAutofillAgent::PasswordInfo::PasswordInfo()
     : backspace_pressed_last(false), password_was_edited_last(false) {
 }
 
 bool PasswordAutofillAgent::ShowSuggestionPopup(
     const PasswordFormFillData& fill_data,
     const blink::WebInputElement& user_input,
@@ skipping 176 matching lines @@
 void PasswordAutofillAgent::LegacyPasswordAutofillAgent::DidStopLoading() {
   agent_->DidStopLoading();
 }
 
 void PasswordAutofillAgent::LegacyPasswordAutofillAgent::
     DidStartProvisionalLoad(blink::WebLocalFrame* navigated_frame) {
   agent_->LegacyDidStartProvisionalLoad(navigated_frame);
 }
 
 }  // namespace autofill