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

chrome/browser/ui/gtk/password_generation_bubble_gtk.h

Index: chrome/browser/ui/gtk/password_generation_bubble_gtk.h
diff --git a/chrome/browser/ui/gtk/password_generation_bubble_gtk.h b/chrome/browser/ui/gtk/password_generation_bubble_gtk.h
index 1de4e2d205b844b7e4f161eff703ac4a8be5b3da..ac92a329491d7a12ff5682af9f37d17c8a3a50f2 100644
--- a/chrome/browser/ui/gtk/password_generation_bubble_gtk.h
+++ b/chrome/browser/ui/gtk/password_generation_bubble_gtk.h
@@ -8,10 +8,14 @@
 
 #include <gtk/gtk.h>
 
-#include "chrome/browser/autofill/password_generator.h"
+#include "base/basictypes.h"

[Garrett Casto, 2012/06/04 23:50:52]

What is this needed for?

[zysxqn, 2012/06/05 16:18:14]

For DISALLOW_COPY_AND_ASSIGN. Previously it can compile because it's indirectly
included from password_generator.h which is moved now. Also I think the code
style requires to include directly.

 #include "ui/base/gtk/gtk_signal.h"
 #include "ui/gfx/rect.h"
 
+namespace autofill {
+class PasswordGenerator;
+}
+
 namespace content {
 class RenderViewHost;
 }
@@ -28,7 +32,8 @@ class PasswordGenerationBubbleGtk {
   PasswordGenerationBubbleGtk(const gfx::Rect& anchor_rect,
                               GtkWidget* anchor_widget,
                               Profile* profile,
-                              content::RenderViewHost* render_view_host);
+                              content::RenderViewHost* render_view_host,
+                              autofill::PasswordGenerator* password_generator);
   virtual ~PasswordGenerationBubbleGtk();
 
  private:
@@ -44,8 +49,8 @@ class PasswordGenerationBubbleGtk {
   // RenderViewHost associated with the button that spawned this bubble.
   content::RenderViewHost* render_view_host_;
 
-  // Class that deals with generating passwords.
-  autofill::PasswordGenerator password_generator_;
+  // Object that deals with generating passwords.
+  autofill::PasswordGenerator* password_generator_;
 
   DISALLOW_COPY_AND_ASSIGN(PasswordGenerationBubbleGtk);
 };