Introduce AutofillClient and use it to get rid of PasswordManager dependency.

chrome/browser/autofill/autofill_manager.h

Index: chrome/browser/autofill/autofill_manager.h
diff --git a/chrome/browser/autofill/autofill_manager.h b/chrome/browser/autofill/autofill_manager.h
index f2db3cef0d8a0b9ab9d988f81ffa1fa0e62ff732..bc5bb95c11cd03173629caf0c0660c1f1245d8dc 100644
--- a/chrome/browser/autofill/autofill_manager.h
+++ b/chrome/browser/autofill/autofill_manager.h
@@ -41,6 +41,7 @@ class TabContents;
 struct ViewHostMsg_FrameNavigate_Params;
 
 namespace autofill {
+class AutofillManagerDelegate;
 class PasswordGenerator;
 }
 
@@ -73,7 +74,10 @@ class AutofillManager : public content::NotificationObserver,
                         public ProfileSyncServiceObserver,
                         public base::RefCounted<AutofillManager> {
  public:
-  explicit AutofillManager(TabContents* tab_contents);
+  // Lifetime of |client| and |tab_contents| must exceed lifetime of
+  // AutofillManager.
+  explicit AutofillManager(autofill::AutofillManagerDelegate* delegate,
+                           TabContents* tab_contents);
 
   // Registers our Enable/Disable Autofill pref.
   static void RegisterUserPrefs(PrefServiceBase* prefs);
@@ -117,7 +121,8 @@ class AutofillManager : public content::NotificationObserver,
   typedef std::pair<std::string, size_t> GUIDPair;
 
   // Test code should prefer to use this constructor.
-  AutofillManager(TabContents* tab_contents,
+  AutofillManager(autofill::AutofillManagerDelegate* delegate,
+                  TabContents* tab_contents,
                   PersonalDataManager* personal_data);
 
   // Returns the value of the AutofillEnabled pref.
@@ -169,6 +174,8 @@ class AutofillManager : public content::NotificationObserver,
   bool OnFormSubmitted(const webkit::forms::FormData& form,
                        const base::TimeTicks& timestamp);
 
+  autofill::AutofillManagerDelegate* const delegate_;

[Ilya Sherman, 2012/08/22 21:47:59]

Note that there is also a member named |external_delegate_|, so this name is
somewhat confusing in context.  It would be nice to rationalize the two
somehow...

[Jói, 2012/08/23 09:50:40]

Renamed this one manager_delegate_.

+
  private:
   // content::WebContentsObserver:
   virtual void RenderViewCreated(content::RenderViewHost* host) OVERRIDE;