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

Unified Diff: chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc

Issue 328113003: Revert "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/views/passwords/manage_passwords_icon_view.cc
diff --git a/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc b/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
index 42457d001978e73f58b838eedd5bac30851dd331..72e3002b82f6fff331a9bf5e83e80b8e8f29a85b 100644
--- a/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
+++ b/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
@@ -41,11 +41,10 @@ void ManagePasswordsIconView::UpdateVisibleUI() {
// Otherwise, start with the correct values for MANAGE_STATE, and adjust
// things accordingly if we're either in BLACKLIST_STATE or PENDING_STATE.
- icon_id_ = active() ? IDR_SAVE_PASSWORD_ACTIVE : IDR_SAVE_PASSWORD_INACTIVE;
+ icon_id_ = IDR_SAVE_PASSWORD;
tooltip_text_id_ = IDS_PASSWORD_MANAGER_TOOLTIP_MANAGE;
if (state() == password_manager::ui::BLACKLIST_STATE)
- icon_id_ = active() ? IDR_SAVE_PASSWORD_DISABLED_ACTIVE
- : IDR_SAVE_PASSWORD_DISABLED_INACTIVE;
+ icon_id_ = IDR_SAVE_PASSWORD_BLACKLISTED;
else if (password_manager::ui::IsPendingState(state()))
tooltip_text_id_ = IDS_PASSWORD_MANAGER_TOOLTIP_SAVE;

Powered by Google App Engine
This is Rietveld 408576698