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

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

Issue 2265133003: Fix warning message of the user profile is not seen on second instance (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 8ba4ac4b55f98a04e557da1acaa83c644849ac4f..498971461d956f266797041110245b98579baccf 100644
--- a/ui/login/account_picker/user_pod_row.js
+++ b/ui/login/account_picker/user_pod_row.js
@@ -1533,8 +1533,6 @@ cr.define('login', function() {
this.actionBoxMenuRemoveElement.hidden = true;
this.actionBoxRemoveUserWarningElement.hidden = false;
- // Show extra statistics information for desktop users
- var message;
if (!this.user.isDesktopUser) {
this.moveActionMenuUpIfNeeded_();
if (!this.user.legacySupervisedUser) {
@@ -1544,12 +1542,11 @@ cr.define('login', function() {
'.action-box-remove-user-warning-table-nonsync').style.display
= 'none';
var message = loadTimeData.getString('removeNonOwnerUserWarningText');
- var element_id = '.action-box-remove-non-owner-user-warning-text';
- this.updateRemoveWarningDialogSetMessage_(element_id,
- this.user.profilePath,
- message);
+ this.updateRemoveNonOwnerUserWarningMessage_(this.user.profilePath,
+ message);
}
} else {
+ // Show extra statistics information for desktop users
this.querySelector(
'.action-box-remove-non-owner-user-warning-text').hidden = true;
this.RemoveWarningDialogSetMessage_(true, false);
@@ -1612,7 +1609,6 @@ cr.define('login', function() {
if (total_count)
this.classList.remove('has-no-stats');
- var elementSelector = '.action-box-remove-user-warning-text';
var is_synced_user = this.user.emailAddress !== "";
// Write total number if all statistics are loaded.
if (num_stats_loaded === Object.keys(stats_elements).length) {
@@ -1621,8 +1617,7 @@ cr.define('login', function() {
var message = loadTimeData.getString(
is_synced_user ? 'removeUserWarningTextSyncNoStats' :
'removeUserWarningTextNonSyncNoStats');
- this.updateRemoveWarningDialogSetMessage_(elementSelector,
- this.user.profilePath,
+ this.updateRemoveWarningDialogSetMessage_(this.user.profilePath,
message);
} else {
window.updateRemoveWarningDialogSetMessage =
@@ -1639,8 +1634,7 @@ cr.define('login', function() {
message = loadTimeData.getString(
is_synced_user ? 'removeUserWarningTextSyncNoStats' :
'removeUserWarningTextNonSyncNoStats');
- this.updateRemoveWarningDialogSetMessage_(elementSelector,
- this.user.profilePath,
+ this.updateRemoveWarningDialogSetMessage_(this.user.profilePath,
message);
} else {
message = loadTimeData.getString(
@@ -1648,8 +1642,7 @@ cr.define('login', function() {
'removeUserWarningTextNonSyncCalculating');
substitute = loadTimeData.getString(
'removeUserWarningTextCalculating');
- this.updateRemoveWarningDialogSetMessage_(elementSelector,
- this.user.profilePath,
+ this.updateRemoveWarningDialogSetMessage_(this.user.profilePath,
message, substitute);
}
}
@@ -1657,22 +1650,19 @@ cr.define('login', function() {
/**
* Refresh the message in the remove user warning dialog.
- * @param {string} elementSelector The elementSelector of warning dialog.
* @param {string} profilePath The filepath of the URL (must be verified).
* @param {string} message The message to be written.
* @param {number|string=} count The number or string to replace $1 in
* |message|. Can be omitted if $1 is not present in |message|.
*/
- updateRemoveWarningDialogSetMessage_: function(elementSelector,
- profilePath,
- message,
+ updateRemoveWarningDialogSetMessage_: function(profilePath, message,
count) {
if (profilePath !== this.user.profilePath)
return;
// Add localized messages where $1 will be replaced with
// <span class="total-count"></span> and $2 will be replaced with
// <span class="email"></span>.
- var element = this.querySelector(elementSelector);
+ var element = this.querySelector('.action-box-remove-user-warning-text');
element.textContent = '';
messageParts = message.split(/(\$[12])/);
@@ -1696,6 +1686,35 @@ cr.define('login', function() {
},
/**
+ * Update the message in the remove non-owner user warning dialog on CrOS.
Alexander Alekseev 2016/08/24 01:02:56 nit: Update message in the "remove non-owner user
+ * @param {string} profilePath The filepath of the URL (must be verified).
+ * @param (string) message The message to be written.
+ */
+ updateRemoveNonOwnerUserWarningMessage_: function(profilePath, message) {
+ if (profilePath !== this.user.profilePath)
+ return;
+ // Add localized messages where $1 will be replaced with
+ // <span class="email"></span>.
+ var element = this.querySelector(
+ '.action-box-remove-non-owner-user-warning-text');
+ element.textContent = '';
+
+ messageParts = message.split(/(\$[1])/);
+ var numParts = messageParts.length;
+ for (var j = 0; j < numParts; j++) {
+ if (messageParts[j] == '$1') {
+ var elementToAdd = document.createElement('span');
+ elementToAdd.classList.add('email');
+ elementToAdd.textContent = this.user.emailAddress;
+ element.appendChild(elementToAdd);
+ } else {
+ element.appendChild(document.createTextNode(messageParts[j]));
+ }
+ }
+ this.moveActionMenuUpIfNeeded_();
+ },
+
+ /**
* Handles a click event on remove user confirmation button.
* @param {Event} e Click event.
*/
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698