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

Unified Diff: chrome/browser/ui/views/autofill/decorated_textfield.h

Issue 55243005: Implement learn more bubble on views. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge Created 7 years, 1 month 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/autofill/decorated_textfield.h
diff --git a/chrome/browser/ui/views/autofill/decorated_textfield.h b/chrome/browser/ui/views/autofill/decorated_textfield.h
index b14798a46629d14e84e1d4f5c4a8a515ea982359..b936b9c5451243a52837aa274b97254d551ed7c7 100644
--- a/chrome/browser/ui/views/autofill/decorated_textfield.h
+++ b/chrome/browser/ui/views/autofill/decorated_textfield.h
@@ -6,6 +6,7 @@
#define CHROME_BROWSER_UI_VIEWS_AUTOFILL_DECORATED_TEXTFIELD_H_
#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
#include "base/strings/string16.h"
#include "ui/gfx/image/image.h"
#include "ui/views/controls/textfield/textfield.h"
@@ -18,6 +19,8 @@ class TextfieldController;
namespace autofill {
+class LearnMoreIconDelegate;
+
// A class which holds a textfield and draws extra stuff on top, like
// invalid content indications.
class DecoratedTextfield : public views::Textfield {
@@ -41,9 +44,10 @@ class DecoratedTextfield : public views::Textfield {
// text.
void SetIcon(const gfx::Image& icon);
- // Sets a tooltip for this field. This will override the icon set with
+ // Sets a learn more icon for this field. This will override the icon set with
// SetIcon(), if any, and will be overridden by future calls to SetIcon().
- void SetTooltipIcon(const base::string16& text);
+ void SetLearnMoreIcon(const base::string16& text,
+ const base::WeakPtr<LearnMoreIconDelegate>& delegate);
// views::Textfield implementation.
virtual base::string16 GetPlaceholderText() const OVERRIDE;
@@ -64,7 +68,7 @@ class DecoratedTextfield : public views::Textfield {
// necessary for the sake of the padding around the native textfield.
void UpdateBackground();
- // Called to update the layout after SetIcon or SetTooltipIcon has been
+ // Called to update the layout after SetIcon or SetLearnMoreIcon has been
// called.
void IconChanged();

Powered by Google App Engine
This is Rietveld 408576698