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

Unified Diff: chrome/browser/ui/passwords/manage_passwords_icon.h

Issue 324853004: Password bubble: Add an "active" icon state, and a new disabled icon. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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/passwords/manage_passwords_icon.h
diff --git a/chrome/browser/ui/passwords/manage_passwords_icon.h b/chrome/browser/ui/passwords/manage_passwords_icon.h
index 76daed5bcd279d2832ed04d0c1b58c971e7c1fcc..74f8d5ba3b93b0791c81b1d669e2d134b84a156e 100644
--- a/chrome/browser/ui/passwords/manage_passwords_icon.h
+++ b/chrome/browser/ui/passwords/manage_passwords_icon.h
@@ -17,6 +17,9 @@ class ManagePasswordsIcon {
void SetState(password_manager::ui::State state);
password_manager::ui::State state() const { return state_; }
+ void SetActive(bool active);
vabr (Chromium) 2014/06/10 12:51:11 nit: In addition to the CamelCase indicating this
+ bool active() const { return active_; }
+
protected:
ManagePasswordsIcon();
~ManagePasswordsIcon();
@@ -27,6 +30,7 @@ class ManagePasswordsIcon {
private:
password_manager::ui::State state_;
+ bool active_;
DISALLOW_COPY_AND_ASSIGN(ManagePasswordsIcon);
};

Powered by Google App Engine
This is Rietveld 408576698