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

chrome/browser/password_manager/content_password_manager_driver.h

Index: chrome/browser/password_manager/content_password_manager_driver.h
diff --git a/chrome/browser/password_manager/content_password_manager_driver.h b/chrome/browser/password_manager/content_password_manager_driver.h
index 875672b364f8f709441b0f180c799b4d2bf2b7a3..2778c38a7ec91cbad3f5dc32d9364d92a9d7f9b9 100644
--- a/chrome/browser/password_manager/content_password_manager_driver.h
+++ b/chrome/browser/password_manager/content_password_manager_driver.h
@@ -7,6 +7,7 @@
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.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_driver.h"
@@ -14,6 +15,7 @@
 
 namespace autofill {
 class AutofillManager;
+class AutofillManagerDelegate;

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

Is this necessary?

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

Fixed.

 struct PasswordForm;
 }
 
@@ -24,22 +26,28 @@ class WebContents;
 class ContentPasswordManagerDriver : public PasswordManagerDriver,
                                      public content::WebContentsObserver {
  public:
-  ContentPasswordManagerDriver(content::WebContents* web_contents,
-                               PasswordManagerClient* client);
+  ContentPasswordManagerDriver(
+      content::WebContents* web_contents,
+      PasswordManagerClient* client);
   virtual ~ContentPasswordManagerDriver();
 
   // PasswordManagerDriver implementation.
   virtual void FillPasswordForm(const autofill::PasswordFormFillData& form_data)
       OVERRIDE;
+  virtual bool DidLastPageLoadEncounterSSLErrors() OVERRIDE;
+  virtual bool IsOffTheRecord() OVERRIDE;
   virtual void AllowPasswordGenerationForForm(autofill::PasswordForm* form)
       OVERRIDE;
   virtual void AccountCreationFormsFound(
       const std::vector<autofill::FormData>& forms) OVERRIDE;
-  virtual bool DidLastPageLoadEncounterSSLErrors() OVERRIDE;
-  virtual bool IsOffTheRecord() OVERRIDE;
+  virtual void AcceptPasswordAutofillSuggestion(
+      const base::string16& username,
+      const base::string16& password) OVERRIDE;
+
   virtual PasswordGenerationManager* GetPasswordGenerationManager() OVERRIDE;
   virtual PasswordManager* GetPasswordManager() OVERRIDE;
   virtual autofill::AutofillManager* GetAutofillManager() OVERRIDE;
+  virtual PasswordAutofillManager* GetPasswordAutofillManager() OVERRIDE;
 
   // content::WebContentsObserver overrides.
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
@@ -50,6 +58,7 @@ class ContentPasswordManagerDriver : public PasswordManagerDriver,
  private:
   PasswordManager password_manager_;
   PasswordGenerationManager password_generation_manager_;
+  PasswordAutofillManager password_autofill_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentPasswordManagerDriver);
 };