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

chrome/browser/ui/browser_window.h

Index: chrome/browser/ui/browser_window.h
diff --git a/chrome/browser/ui/browser_window.h b/chrome/browser/ui/browser_window.h
index 508249ae1f66626461f7897e58cfe4fbc0d72290..d5eb2f9f7681aed35b1f37f2d53afbd6b8c82737 100644
--- a/chrome/browser/ui/browser_window.h
+++ b/chrome/browser/ui/browser_window.h
@@ -29,6 +29,9 @@ class TemplateURL;
 class ToolbarView;
 #endif
 
+namespace autofill {
+class PasswordGenerator;
+}
 namespace content {
 class WebContents;
 struct NativeWebKeyboardEvent;
@@ -361,7 +364,9 @@ class BrowserWindow : public BaseWindow {
   // Show bubble for password generation positioned relative to |rect|. A stub
   // implementation is provided since this feature is currently only available
   // for Windows.
-  virtual void ShowPasswordGenerationBubble(const gfx::Rect& rect) {}
+  virtual void ShowPasswordGenerationBubble(
+      const gfx::Rect& rect,
+      autofill::PasswordGenerator* password_generator) {}

[Ilya Sherman, 2012/06/04 21:17:52]

This default implementation should be careful not to leak the
|password_generator| if you decide on a transfer-of-ownership semantics.

[zysxqn, 2012/06/04 22:40:15]

NA now.

 
  protected:
   friend void browser::CloseAllBrowsers();