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

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_GTK_PASSWORD_GENERATION_BUBBLE_GTK_H_
 #define CHROME_BROWSER_UI_GTK_PASSWORD_GENERATION_BUBBLE_GTK_H_
 #pragma once
 
 #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;
 }
 
 class BubbleGtk;
 class Profile;
 
 // PasswordGenerationBubbleGtk is a bubble use to show possible generated
 // passwords to users. It is set in page content, anchored at |anchor_rect|.
 // If the generated password is accepted by the user, the renderer associated
 // with |render_view_host| is informed of this password.
 class PasswordGenerationBubbleGtk {
  public:
   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:
   CHROMEGTK_CALLBACK_0(PasswordGenerationBubbleGtk, void, OnDestroy);
   CHROMEGTK_CALLBACK_0(PasswordGenerationBubbleGtk, void, OnAcceptClicked);
   CHROMEG_CALLBACK_0(
       PasswordGenerationBubbleGtk, void, OnLearnMoreLinkClicked, GtkButton*);
 
   BubbleGtk* bubble_;
   GtkWidget* text_field_;
   Profile* profile_;
 
   // 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);
 };
 
 #endif  // CHROME_BROWSER_UI_GTK_PASSWORD_GENERATION_BUBBLE_GTK_H_