Chromium Code Reviews| 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 0f854e0ba07a820dc14601dd4c059926d1345f2e..1bd2b8c6bf4373dfa14696c9cd9636136a4bb5e3 100644 |
| --- a/ui/login/account_picker/user_pod_row.js |
| +++ b/ui/login/account_picker/user_pod_row.js |
| @@ -2574,14 +2574,30 @@ cr.define('login', function() { |
| }, |
| /** |
| + * Toggles pod PIN keyboard visiblity. |
| + * @param {!user} username |
| + * @param {boolean} visible |
| + */ |
| + setPinVisibility: function(username, visible) { |
| + var pod = this.getPodWithUsername_(username); |
| + if (!pod) { |
| + console.warn('Attempt to change pin visibility to ' + visible + |
| + ' for missing pod ' + JSON.stringify(username) + '.'); |
|
xiyuan
2016/06/29 21:53:35
Don't dump user name in log. It is not privacy fri
jdufault
2016/06/29 22:05:32
Done.
|
| + return; |
| + } |
| + |
| + pod.setPinVisibility(visible); |
| + }, |
| + |
| + /** |
| * Removes user pod from pod row. |
| - * @param {string} email User's email. |
| + * @param {!user} username |
| */ |
| removeUserPod: function(username) { |
| var podToRemove = this.getPodWithUsername_(username); |
| if (podToRemove == null) { |
| - console.warn('Attempt to remove not existing pod for ' + username + |
| - '.'); |
| + console.warn('Attempt to remove not existing pod for ' + |
| + JSON.stringify(username) + '.'); |
|
xiyuan
2016/06/29 21:53:35
ditto
jdufault
2016/06/29 22:05:32
Done.
|
| return; |
| } |
| this.removeChild(podToRemove); |