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

Unified Diff: ui/login/account_picker/user_pod_row.js

Issue 528423002: Unload Easy Unlock app when ChromeOS suspends and reload on wake up. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Unload instead of uninstalling app Created 6 years, 3 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: ui/login/account_picker/user_pod_row.js
diff --git a/ui/login/account_picker/user_pod_row.js b/ui/login/account_picker/user_pod_row.js
index 356a1a61fd814cc8e5a1c72184d5d66be5262431..f3b9ed55613c5a725f0e31ba6e445db0edd2a06d 100644
--- a/ui/login/account_picker/user_pod_row.js
+++ b/ui/login/account_picker/user_pod_row.js
@@ -345,7 +345,7 @@ cr.define('login', function() {
this.hideTooltip_(true);
this.iconElement.classList.add('faded');
- this.hideTransitionListener_ = this.hide_.bind(this);
+ this.hideTransitionListener_ = this.hide.bind(this);
this.iconElement.addEventListener('webkitTransitionEnd',
this.hideTransitionListener_);
ensureTransitionEndEvent(this.iconElement, 200);
@@ -467,9 +467,8 @@ cr.define('login', function() {
/**
* Hides the icon. Makes sure the tooltip is hidden and animation reset.
- * @private
*/
- hide_: function() {
+ hide: function() {
this.hideTooltip_(true);
this.hidden = true;
this.setAnimation(null);
@@ -2358,7 +2357,8 @@ cr.define('login', function() {
return;
}
- pod.customIconElement.fadeOut();
+ // TODO(tengs): Allow option for a fading transition.
+ pod.customIconElement.hide();
},
/**
« chrome/browser/signin/easy_unlock_service.cc ('K') | « chrome/browser/signin/easy_unlock_service.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698