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

Unified Diff: chrome/browser/resources/settings/date_time_page/date_time_page.js

Issue 2447343002: MD Settings: Show "Set date and time" button when relevant (Closed)
Patch Set: paper-icon-button-light.html 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/settings/date_time_page/date_time_page.js
diff --git a/chrome/browser/resources/settings/date_time_page/date_time_page.js b/chrome/browser/resources/settings/date_time_page/date_time_page.js
index ed3992bbfa0345213cfe6c50462335e62a5bc539..5d1989436ace525b6e19ff3e3cedd45a1ae10b9e 100644
--- a/chrome/browser/resources/settings/date_time_page/date_time_page.js
+++ b/chrome/browser/resources/settings/date_time_page/date_time_page.js
@@ -100,8 +100,10 @@ Polymer({
this.addWebUIListener(
'time-zone-auto-detect-policy',
this.onTimeZoneAutoDetectPolicyChanged_.bind(this));
- chrome.send('dateTimePageReady');
+ this.addWebUIListener(
+ 'can-set-date-time-changed', this.onCanSetDateTimeChanged_.bind(this));
+ chrome.send('dateTimePageReady');
this.maybeGetTimeZoneList_();
},
@@ -134,6 +136,14 @@ Polymer({
},
/**
+ * @param {boolean} canSetDateTime Whether date and time are settable.
+ * @private
+ */
+ onCanSetDateTimeChanged_: function(canSetDateTime) {
+ this.$.setDateTime.classList.toggle('hidden', !canSetDateTime);
+ },
+
+ /**
* @param {!Event} e
* @private
*/
@@ -142,6 +152,11 @@ Polymer({
'settings.resolve_timezone_by_geolocation', e.target.checked);
},
+ /** @private */
+ onSetDateTimeTap_: function() {
+ chrome.send('showSetDateTimeUI');
+ },
+
/**
* @param {settings.TimeZoneAutoDetectPolicy} timeZoneAutoDetectPolicy
* @return {boolean}

Powered by Google App Engine
This is Rietveld 408576698