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

Unified Diff: chrome/browser/resources/options/browser_options.js

Issue 2393643004: [MD Settings][People] Updates positions of username and sync status message (Closed)
Patch Set: Created 4 years, 2 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: chrome/browser/resources/options/browser_options.js
diff --git a/chrome/browser/resources/options/browser_options.js b/chrome/browser/resources/options/browser_options.js
index 684c40b81ee0b86a46e5b1472bd45b06a6615cbf..4d54bd690e392d0811980e738797feb4b839867f 100644
--- a/chrome/browser/resources/options/browser_options.js
+++ b/chrome/browser/resources/options/browser_options.js
@@ -17,6 +17,7 @@ cr.exportPath('options');
* signoutAllowed: (boolean|undefined),
* statusText: (string|undefined),
* supervisedUser: (boolean|undefined),
+ * syncAccountInfo: (string|undefined),
maxbogue 2016/10/05 15:56:23 Why syncAccountInfo and not just accountInfo?
Moe 2016/10/05 18:52:45 Done.
* syncSystemEnabled: (boolean|undefined)}}
* @see chrome/browser/ui/webui/options/browser_options_handler.cc
*/
@@ -1188,6 +1189,9 @@ cr.define('options', function() {
loadTimeData.getString('syncButtonTextSignIn');
$('start-stop-sync-indicator').hidden = signInButton.hidden;
+ $('sync-account-info').textContent = syncData.syncAccountInfo;
+ $('sync-account-info').hidden = !this.signedIn_;
+
// TODO(estade): can this just be textContent?
$('sync-status-text').innerHTML = syncData.statusText;
var statusSet = syncData.statusText.length != 0;

Powered by Google App Engine
This is Rietveld 408576698