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

components/password_manager/core/browser/password_generation_manager_unittest.cc

 // Copyright 2014 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 <vector>
 
 #include "base/prefs/pref_registry_simple.h"
 #include "base/prefs/pref_service.h"
 #include "base/prefs/testing_pref_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/browser/autofill_field.h"
 #include "components/autofill/core/browser/autofill_metrics.h"
 #include "components/autofill/core/browser/form_structure.h"
 #include "components/autofill/core/common/form_data.h"
 #include "components/autofill/core/common/form_field_data.h"
+#include "components/password_manager/core/browser/password_autofill_manager.h"
 #include "components/password_manager/core/browser/password_generation_manager.h"
 #include "components/password_manager/core/browser/password_manager.h"
 #include "components/password_manager/core/browser/password_manager_client.h"
 #include "components/password_manager/core/common/password_manager_pref_names.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 using base::ASCIIToUTF16;
 
 namespace {
 
 class TestPasswordManagerDriver : public PasswordManagerDriver {
  public:
   TestPasswordManagerDriver(PasswordManagerClient* client)
       : password_manager_(client),
         password_generation_manager_(client),
+        password_autofill_manager_(client, NULL),
         is_off_the_record_(false) {}
   virtual ~TestPasswordManagerDriver() {}
 
   // PasswordManagerDriver implementation.
   virtual void FillPasswordForm(const autofill::PasswordFormFillData& form_data)
       OVERRIDE {}
   virtual bool DidLastPageLoadEncounterSSLErrors() OVERRIDE { return false; }
   virtual bool IsOffTheRecord() OVERRIDE { return is_off_the_record_; }
   virtual PasswordGenerationManager* GetPasswordGenerationManager() OVERRIDE {
     return &password_generation_manager_;
   }
   virtual PasswordManager* GetPasswordManager() OVERRIDE {
     return &password_manager_;
   }
   virtual autofill::AutofillManager* GetAutofillManager() OVERRIDE {
     return NULL;
   }
+  virtual PasswordAutofillManager* GetPasswordAutofillManager() OVERRIDE {
+    return &password_autofill_manager_;
+  }
   virtual void AllowPasswordGenerationForForm(autofill::PasswordForm* form)
       OVERRIDE {}
   virtual void AccountCreationFormsFound(
       const std::vector<autofill::FormData>& forms) OVERRIDE {
     found_account_creation_forms_.insert(
         found_account_creation_forms_.begin(), forms.begin(), forms.end());
   }
+  virtual void AcceptPasswordAutofillSuggestion(
+      const base::string16& username,
+      const base::string16& password) OVERRIDE {
+  }

[Ilya Sherman, 2014/03/18 00:14:27]

nit: Do you need to add these methods?  It doesn't look like they're ever used.

[Patrick Dubroy, 2014/03/28 15:44:22]

See previous comment.

 
   const std::vector<autofill::FormData>& GetFoundAccountCreationForms() {
     return found_account_creation_forms_;
   }
   void set_is_off_the_record(bool is_off_the_record) {
     is_off_the_record_ = is_off_the_record;
   }
 
  private:
   PasswordManager password_manager_;
   PasswordGenerationManager password_generation_manager_;
+  PasswordAutofillManager password_autofill_manager_;
   std::vector<autofill::FormData> found_account_creation_forms_;
   bool is_off_the_record_;
 };
 
 class TestPasswordManagerClient : public PasswordManagerClient {
  public:
   TestPasswordManagerClient(scoped_ptr<PrefService> prefs)
       : prefs_(prefs.Pass()), driver_(this), is_sync_enabled_(false) {}
 
   virtual void PromptUserToSavePassword(PasswordFormManager* form_to_save)
@@ skipping 134 matching lines @@
   // Disable password manager by going incognito. Even though password
   // syncing is enabled, generation should still
   // be disabled.
   GetTestDriver()->set_is_off_the_record(true);
   PrefService* prefs = client_->GetPrefs();
   prefs->SetBoolean(prefs::kPasswordManagerEnabled, true);
   client_->set_is_password_sync_enabled(true);
 
   EXPECT_FALSE(IsGenerationEnabled());
 }