Abstracted AcceptPasswordAutofillSuggestion IPC out of core Autofill code.

components/autofill/core/browser/password_autofill_manager_unittest.cc

Index: components/autofill/core/browser/password_autofill_manager_unittest.cc
diff --git a/components/autofill/core/browser/password_autofill_manager_unittest.cc b/components/autofill/core/browser/password_autofill_manager_unittest.cc
index 96ce202e614f5f935f097e841c876913abf8568f..2df433639c3bf163f16b344282fd422de877038e 100644
--- a/components/autofill/core/browser/password_autofill_manager_unittest.cc
+++ b/components/autofill/core/browser/password_autofill_manager_unittest.cc
@@ -69,4 +69,8 @@ TEST_F(PasswordAutofillManagerTest, DidAcceptAutofillSuggestion) {
       username_field(), ASCIIToUTF16(kAliceUsername)));
 }
 
+// TODO(jeanfrancoisg): Add unittest checking that AutofillDriver's

[blundell, 2013/11/20 20:32:43]

I think that you could just make a MockAutofillDriver that subclasses
TestAutofillDriver and passes in NULL to the TestAutofillDriver constructor,
since the only thing that you want to do is expect that the
RenderShouldAcceptPasswordAutofillSuggestion() method is called.

[jif-google, 2013/11/21 16:46:02]

Done.

+// RendererShouldAcceptPasswordAutofillSuggestion is called when calling
+// DidAcceptAutofillSuggestion. http://crbug.com/321721
+
 }  // namespace autofill