Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 /** | 5 /** |
| 6 * @fileoverview User pod row implementation. | 6 * @fileoverview User pod row implementation. |
| 7 */ | 7 */ |
| 8 | 8 |
| 9 cr.define('login', function() { | 9 cr.define('login', function() { |
| 10 /** | 10 /** |
| (...skipping 2333 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 2344 * @param {string} username Username of pod to remove button | 2344 * @param {string} username Username of pod to remove button |
| 2345 */ | 2345 */ |
| 2346 hideUserPodCustomIcon: function(username) { | 2346 hideUserPodCustomIcon: function(username) { |
| 2347 var pod = this.getPodWithUsername_(username); | 2347 var pod = this.getPodWithUsername_(username); |
| 2348 if (pod == null) { | 2348 if (pod == null) { |
| 2349 console.error('Unable to hide user pod button for ' + username + | 2349 console.error('Unable to hide user pod button for ' + username + |
| 2350 ': user pod not found.'); | 2350 ': user pod not found.'); |
| 2351 return; | 2351 return; |
| 2352 } | 2352 } |
| 2353 | 2353 |
| 2354 pod.customIconElement.fadeOut(); | 2354 // TODO(tengs): Allow option for a fading transition. |
| 2355 pod.customIconElement.hidden = true; | |
|
tbarzic
2014/09/03 21:40:37
why is this needed?
Tim Song
2014/09/03 21:53:39
Changed to use hide() instead.
| |
| 2355 }, | 2356 }, |
| 2356 | 2357 |
| 2357 /** | 2358 /** |
| 2358 * Sets the authentication type used to authenticate the user. | 2359 * Sets the authentication type used to authenticate the user. |
| 2359 * @param {string} username Username of selected user | 2360 * @param {string} username Username of selected user |
| 2360 * @param {number} authType Authentication type, must be one of the | 2361 * @param {number} authType Authentication type, must be one of the |
| 2361 * values listed in AUTH_TYPE enum. | 2362 * values listed in AUTH_TYPE enum. |
| 2362 * @param {string} value The initial value to use for authentication. | 2363 * @param {string} value The initial value to use for authentication. |
| 2363 */ | 2364 */ |
| 2364 setAuthType: function(username, authType, value) { | 2365 setAuthType: function(username, authType, value) { |
| (...skipping 622 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 2987 if (pod && pod.multiProfilesPolicyApplied) { | 2988 if (pod && pod.multiProfilesPolicyApplied) { |
| 2988 pod.userTypeBubbleElement.classList.remove('bubble-shown'); | 2989 pod.userTypeBubbleElement.classList.remove('bubble-shown'); |
| 2989 } | 2990 } |
| 2990 } | 2991 } |
| 2991 }; | 2992 }; |
| 2992 | 2993 |
| 2993 return { | 2994 return { |
| 2994 PodRow: PodRow | 2995 PodRow: PodRow |
| 2995 }; | 2996 }; |
| 2996 }); | 2997 }); |
| OLD | NEW |