Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10024)

Unified Diff: chrome/browser/ui/views/password_generation_bubble_view.h

Issue 10458018: This CL does the following: (1) Pass the max_length attribute to the password generator; (2) Update… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Futher comments. Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/password_generation_bubble_view.h
diff --git a/chrome/browser/ui/views/password_generation_bubble_view.h b/chrome/browser/ui/views/password_generation_bubble_view.h
index bce9aff6da6c6b192748f5f6917bd898f4e85287..68ce8765ee9936359c352dc71e43e89662c03a02 100644
--- a/chrome/browser/ui/views/password_generation_bubble_view.h
+++ b/chrome/browser/ui/views/password_generation_bubble_view.h
@@ -6,12 +6,16 @@
#define CHROME_BROWSER_UI_VIEWS_PASSWORD_GENERATION_BUBBLE_VIEW_H_
#pragma once
-#include "chrome/browser/autofill/password_generator.h"
+#include "base/basictypes.h"
Garrett Casto 2012/06/04 23:50:52 Same here.
zysxqn 2012/06/05 16:18:14 Also for DISALLOW_COPY_AND_ASSIGN.
#include "ui/gfx/rect.h"
#include "ui/views/bubble/bubble_delegate.h"
#include "ui/views/controls/button/button.h"
#include "ui/views/view.h"
+namespace autofill {
+class PasswordGenerator;
+}
+
namespace content {
class RenderViewHost;
}
@@ -30,7 +34,8 @@ class PasswordGenerationBubbleView : public views::BubbleDelegateView,
public:
PasswordGenerationBubbleView(const gfx::Rect& anchor_rect,
views::View* anchor_view,
- content::RenderViewHost* render_view_host);
+ content::RenderViewHost* render_view_host,
+ autofill::PasswordGenerator* password_generator);
virtual ~PasswordGenerationBubbleView();
private:
@@ -52,8 +57,8 @@ class PasswordGenerationBubbleView : public views::BubbleDelegateView,
// RenderViewHost associated with the button that spawned this bubble.
content::RenderViewHost* render_view_host_;
- // Class to generate passwords
- autofill::PasswordGenerator password_generator_;
+ // Object to generate passwords
+ autofill::PasswordGenerator* password_generator_;
DISALLOW_COPY_AND_ASSIGN(PasswordGenerationBubbleView);
};

Powered by Google App Engine
This is Rietveld 408576698