Autofill: Refactoring to support fetching password after a username is selected

chrome/browser/password_manager/content_password_manager_driver.cc

Index: chrome/browser/password_manager/content_password_manager_driver.cc
diff --git a/chrome/browser/password_manager/content_password_manager_driver.cc b/chrome/browser/password_manager/content_password_manager_driver.cc
index b78dd8319e74162d4039179f7e0ea801654fc585..6f91c68ba44654c1ebd83b2c2e870169b70ebcc4 100644
--- a/chrome/browser/password_manager/content_password_manager_driver.cc
+++ b/chrome/browser/password_manager/content_password_manager_driver.cc
@@ -23,7 +23,8 @@ ContentPasswordManagerDriver::ContentPasswordManagerDriver(
     PasswordManagerClient* client)
     : WebContentsObserver(web_contents),
       password_manager_(client),
-      password_generation_manager_(client) {
+      password_generation_manager_(client),
+      password_autofill_manager_(client, GetAutofillManager()->delegate()) {
   DCHECK(web_contents);
 }
 
@@ -49,6 +50,18 @@ void ContentPasswordManagerDriver::AccountCreationFormsFound(
                                                           forms));
 }
 
+void
+ContentPasswordManagerDriver::AcceptPasswordAutofillSuggestion(

[Garrett Casto, 2014/03/12 01:09:22]

It looks like the "void" could be on the same line.

[Patrick Dubroy, 2014/03/12 16:47:09]

Yeah, not sure what happened. Fixed.

+    const base::string16& username,
+    const base::string16& password) {
+  DCHECK(web_contents());
+  content::RenderViewHost* host = web_contents()->GetRenderViewHost();
+  host->Send(
+      new AutofillMsg_AcceptPasswordAutofillSuggestion(host->GetRoutingID(),
+                                                       username,
+                                                       password));
+}
+
 bool ContentPasswordManagerDriver::DidLastPageLoadEncounterSSLErrors() {
   DCHECK(web_contents());
   content::NavigationEntry* entry =
@@ -75,6 +88,11 @@ PasswordManager* ContentPasswordManagerDriver::GetPasswordManager() {
   return &password_manager_;
 }
 
+PasswordAutofillManager*
+ContentPasswordManagerDriver::GetPasswordAutofillManager() {
+  return &password_autofill_manager_;
+}
+
 void ContentPasswordManagerDriver::DidNavigateMainFrame(
     const content::LoadCommittedDetails& details,
     const content::FrameNavigateParams& params) {
@@ -94,6 +112,12 @@ bool ContentPasswordManagerDriver::OnMessageReceived(
   IPC_MESSAGE_FORWARD(AutofillHostMsg_PasswordFormSubmitted,
                       &password_manager_,
                       PasswordManager::OnPasswordFormSubmitted)
+  IPC_MESSAGE_FORWARD(AutofillHostMsg_ShowPasswordSuggestions,
+                      &password_manager_,
+                      PasswordManager::OnShowPasswordSuggestions)
+  IPC_MESSAGE_FORWARD(AutofillHostMsg_AddPasswordFormMapping,
+                      &password_manager_,
+                      PasswordManager::OnAddPasswordFormMapping)
   IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()