Add Password Autofill Manager to New Autofill

chrome/browser/autofill/autofill_external_delegate.h

Index: chrome/browser/autofill/autofill_external_delegate.h
diff --git a/chrome/browser/autofill/autofill_external_delegate.h b/chrome/browser/autofill/autofill_external_delegate.h
index b3413045783364221bb638a2667de391959e6d68..b57937585641254dafb45d0f1a5fa0cef9ec491c 100644
--- a/chrome/browser/autofill/autofill_external_delegate.h
+++ b/chrome/browser/autofill/autofill_external_delegate.h
@@ -10,8 +10,10 @@
 
 #include "base/compiler_specific.h"
 #include "base/string16.h"
+#include "chrome/browser/autofill/password_autofill_manager.h"
 #include "webkit/forms/form_data.h"
 #include "webkit/forms/form_field.h"
+#include "webkit/forms/password_form_dom_manager.h"
 
 class AutofillManager;
 class TabContentsWrapper;
@@ -57,6 +59,12 @@ class AutofillExternalDelegate {
       const std::vector<string16>& autofill_icons,
       const std::vector<int>& autofill_unique_ids);
 
+  // Set the bounds of the Autofill element being worked with.
+  virtual void SetBounds(const gfx::Rect& bounds) = 0;
+
+  // Show password suggestions in the popup.
+  void ShowPasswordSuggestions(const std::vector<string16>& suggestions);
+
   // Inform the delegate that the text field editing has ended, this is
   // used to help record the metrics of when a new popup is shown.
   void DidEndTextFieldEditing();
@@ -72,6 +80,15 @@ class AutofillExternalDelegate {
   // Hide the Autofill poup.
   virtual void HideAutofillPopup();
 
+  // Returns the delegate to its starting state by removing any page specific
+  // values or settings.
+  void Reset();
+
+  // Inform the Password Manager of a filled form.
+  void AddPasswordFormMapping(
+      const webkit::forms::FormField& form,
+      const webkit::forms::PasswordFormFillData& fill_data);
+
   // Platforms that wish to implement an external Autofill delegate
   // MUST implement this.  The 1st arg is the tab contents that owns
   // this delegate; the second is the Autofill manager owned by the
@@ -111,6 +128,9 @@ class AutofillExternalDelegate {
   TabContentsWrapper* tab_contents_wrapper_;  // weak; owns me.
   AutofillManager* autofill_manager_;  // weak.
 
+  // Password Autofill manager, handles all password-related Autofilling.
+  PasswordAutofillManager password_autofill_manager_;
+
   // The ID of the last request sent for form field Autofill.  Used to ignore
   // out of date responses.
   int autofill_query_id_;
@@ -132,6 +152,9 @@ class AutofillExternalDelegate {
   // The menu index of the "Autofill options..." menu item.
   int suggestions_options_index_;
 
+  // The value to give as the unique id for all password entries.
+  const int kPasswordEntryIds = -2;

[Ilya Sherman, 2012/03/20 00:58:55]

nit: "Ids" -> "Id"

[csharp, 2012/03/21 14:10:31]

Done.

[Ilya Sherman, 2012/03/22 01:20:06]

Methinks you interpreted this nit backward ;)

+
   DISALLOW_COPY_AND_ASSIGN(AutofillExternalDelegate);
 };