Make password generation switched by a preference in chrome settings rather than a command line fla…

chrome/browser/autofill/autofill_external_delegate_unittest.cc

Index: chrome/browser/autofill/autofill_external_delegate_unittest.cc
diff --git a/chrome/browser/autofill/autofill_external_delegate_unittest.cc b/chrome/browser/autofill/autofill_external_delegate_unittest.cc
index 7c6131a7f8a5aa9a53824dee511c7d75eaad8f1d..5a52d61a9ecc97ab74c454a34997a22583593738 100644
--- a/chrome/browser/autofill/autofill_external_delegate_unittest.cc
+++ b/chrome/browser/autofill/autofill_external_delegate_unittest.cc
@@ -56,8 +56,9 @@ class MockAutofillExternalDelegate : public TestAutofillExternalDelegate {
 
 class MockAutofillManager : public AutofillManager {
  public:
-  explicit MockAutofillManager(TabContentsWrapper* tab_contents)
-      : AutofillManager(tab_contents) {}
+  MockAutofillManager(TabContentsWrapper* tab_contents,
+                      PersonalDataManager* personal_data)
+      : AutofillManager(tab_contents, personal_data) {}

[Ilya Sherman, 2012/05/16 18:10:38]

nit: Rather than adding a new parameter for |personal_data|, just pass NULL as
the second argument to the AutofillManager constructor, since that's what all
the tests in this file want.

[zysxqn, 2012/05/16 18:25:03]

Done.

 
   MOCK_METHOD4(OnFillAutofillFormData,
                void(int query_id,
@@ -79,7 +80,9 @@ class AutofillExternalDelegateUnitTest : public TabContentsWrapperTestHarness {
 
   virtual void SetUp() OVERRIDE {
     TabContentsWrapperTestHarness::SetUp();
-    autofill_manager_ = new MockAutofillManager(contents_wrapper());
+    // Force to use the constructor designated for unit test, but we don't
+    // really need personal_data in this test so we pass a NULL pointer.
+    autofill_manager_ = new MockAutofillManager(contents_wrapper(), NULL);
     external_delegate_.reset(new MockAutofillExternalDelegate(
         contents_wrapper(),
         autofill_manager_));