This CL does the following: (1) Pass the max_length attribute to the password generator; (2) Update…

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 b48309766fb7d74bbcfc3e27a16f746c759b093b..3feabe7440d3dce9c053fb110dc5d738f0cda392 100644
--- a/chrome/browser/autofill/autofill_manager.h
+++ b/chrome/browser/autofill/autofill_manager.h
@@ -40,6 +40,10 @@ class TabContentsWrapper;
 
 struct ViewHostMsg_FrameNavigate_Params;
 
+namespace autofill {
+class PasswordGenerator;
+}
+
 namespace content {
 class RenderViewHost;
 }
@@ -93,7 +97,7 @@ class AutofillManager : public content::NotificationObserver,
   void OnDidFillAutofillFormData(const base::TimeTicks& timestamp);
   void OnShowAutofillDialog();
   void OnDidPreviewAutofillFormData();
-  void OnShowPasswordGenerationPopup(const gfx::Rect& bounds);
+  void OnShowPasswordGenerationPopup(const gfx::Rect& bounds, int max_length);
 
   // Remove the credit card or Autofill profile that matches |unique_id|
   // from the database.
@@ -358,6 +362,8 @@ class AutofillManager : public content::NotificationObserver,
   base::WeakPtr<ProfileSyncService> sync_service_;
   // Listens for changes to the 'enabled' state for password generation.
   PrefChangeRegistrar registrar_;
+  // To be passed to the Password Generation UI to generate the password.

[Ilya Sherman, 2012/06/04 22:49:15]

nit: "password generation" probably shouldn't be capitalized, as it's not a
feature name per sé.

[zysxqn, 2012/06/04 23:17:19]

Done.

+  scoped_ptr<autofill::PasswordGenerator> password_generator_;
 
   // Our copy of the form data.
   ScopedVector<FormStructure> form_structures_;