Abstracted AcceptPasswordAutofillSuggestion IPC out of core Autofill code.

components/autofill/core/browser/password_autofill_manager.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/core/browser/autofill_driver.h"
 #include "components/autofill/core/browser/password_autofill_manager.h"
-#include "components/autofill/core/common/autofill_messages.h"
-#include "content/public/browser/render_view_host.h"
-#include "content/public/browser/web_contents.h"
 #include "ui/events/keycodes/keyboard_codes.h"
 
 namespace autofill {
 
 ////////////////////////////////////////////////////////////////////////////////
 // PasswordAutofillManager, public:
 
 PasswordAutofillManager::PasswordAutofillManager(
-    content::WebContents* web_contents) : web_contents_(web_contents) {
+    AutofillDriver* autofill_driver) : autofill_driver_(autofill_driver) {

[Ilya Sherman, 2013/11/22 00:26:29]

nit: Please add DCHECK(autofill_driver) here, since it's currently implicitly
assumed that autofill_driver is non-null; and there's no code in the constructor
that explicitly exercises that assumption.

[jif-google, 2013/11/22 12:21:44]

Done.

 }
 
 PasswordAutofillManager::~PasswordAutofillManager() {
 }
 
 bool PasswordAutofillManager::DidAcceptAutofillSuggestion(
     const FormFieldData& field,
-    const base::string16& value) {
+    const base::string16& username) {
   PasswordFormFillData password;
   if (!FindLoginInfo(field, &password))
     return false;
 
-  if (WillFillUserNameAndPassword(value, password)) {
-    if (web_contents_) {
-      content::RenderViewHost* render_view_host =
-          web_contents_->GetRenderViewHost();
-      render_view_host->Send(new AutofillMsg_AcceptPasswordAutofillSuggestion(
-          render_view_host->GetRoutingID(),
-          value));
-    }
+  if (WillFillUserNameAndPassword(username, password)) {
+    autofill_driver_->RendererShouldAcceptPasswordAutofillSuggestion(username);
     return true;
   }
 
   return false;
 }
 
 void PasswordAutofillManager::AddPasswordFormMapping(
     const FormFieldData& username_element,
     const PasswordFormFillData& password) {
   login_to_password_info_[username_element] = password;
@@ skipping 39 matching lines @@
     PasswordFormFillData* found_password) {
   LoginToPasswordInfoMap::iterator iter = login_to_password_info_.find(field);
   if (iter == login_to_password_info_.end())
     return false;
 
   *found_password = iter->second;
   return true;
 }
 
 }  // namespace autofill