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

Side by Side Diff: ui/login/account_picker/user_pod_row.js

Issue 1016113002: Instrument "Remove this person" in user manager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address feedback/Naming tweaks Created 5 years, 9 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 unified diff | Download patch
OLDNEW
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 1397 matching lines...) Expand 10 before | Expand all | Expand 10 after
1408 1408
1409 // Move up the menu if it overlaps shelf. 1409 // Move up the menu if it overlaps shelf.
1410 var maxHeight = cr.ui.LoginUITools.getMaxHeightBeforeShelfOverlapping( 1410 var maxHeight = cr.ui.LoginUITools.getMaxHeightBeforeShelfOverlapping(
1411 this.actionBoxMenu); 1411 this.actionBoxMenu);
1412 var actualHeight = parseInt( 1412 var actualHeight = parseInt(
1413 window.getComputedStyle(this.actionBoxMenu).height); 1413 window.getComputedStyle(this.actionBoxMenu).height);
1414 if (maxHeight < actualHeight) { 1414 if (maxHeight < actualHeight) {
1415 this.actionBoxMenu.classList.add('menu-moved-up'); 1415 this.actionBoxMenu.classList.add('menu-moved-up');
1416 this.actionBoxAreaElement.classList.add('menu-moved-up'); 1416 this.actionBoxAreaElement.classList.add('menu-moved-up');
1417 } 1417 }
1418 chrome.send('logRemoveUserWarningShown');
1418 }, 1419 },
1419 1420
1420 /** 1421 /**
1421 * Handles a click event on remove user confirmation button. 1422 * Handles a click event on remove user confirmation button.
1422 * @param {Event} e Click event. 1423 * @param {Event} e Click event.
1423 */ 1424 */
1424 handleRemoveUserConfirmationClick_: function(e) { 1425 handleRemoveUserConfirmationClick_: function(e) {
1425 if (this.isActionBoxMenuActive) { 1426 if (this.isActionBoxMenuActive) {
1426 this.isActionBoxMenuActive = false; 1427 this.isActionBoxMenuActive = false;
1427 this.removeUser(this.user); 1428 this.removeUser(this.user);
(...skipping 1723 matching lines...) Expand 10 before | Expand all | Expand 10 after
3151 if (pod && pod.multiProfilesPolicyApplied) { 3152 if (pod && pod.multiProfilesPolicyApplied) {
3152 pod.userTypeBubbleElement.classList.remove('bubble-shown'); 3153 pod.userTypeBubbleElement.classList.remove('bubble-shown');
3153 } 3154 }
3154 } 3155 }
3155 }; 3156 };
3156 3157
3157 return { 3158 return {
3158 PodRow: PodRow 3159 PodRow: PodRow
3159 }; 3160 };
3160 }); 3161 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698