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

Side by Side Diff: chrome/browser/resources/settings/date_time_page/date_time_page.js

Issue 2847963003: [MD settings] specify @override on created, ready, and attached (Closed)
Patch Set: Created 3 years, 7 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 6 * @fileoverview
7 * 'settings-date-time-page' is the settings page containing date and time 7 * 'settings-date-time-page' is the settings page containing date and time
8 * settings. 8 * settings.
9 */ 9 */
10 10
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 type: Boolean, 87 type: Boolean,
88 value: false, 88 value: false,
89 }, 89 },
90 }, 90 },
91 91
92 observers: [ 92 observers: [
93 'maybeGetTimeZoneList_(' + 93 'maybeGetTimeZoneList_(' +
94 'prefs.cros.system.timezone.value, timeZoneAutoDetect_)', 94 'prefs.cros.system.timezone.value, timeZoneAutoDetect_)',
95 ], 95 ],
96 96
97 /** @override */
97 attached: function() { 98 attached: function() {
98 this.addWebUIListener( 99 this.addWebUIListener(
99 'time-zone-auto-detect-policy', 100 'time-zone-auto-detect-policy',
100 this.onTimeZoneAutoDetectPolicyChanged_.bind(this)); 101 this.onTimeZoneAutoDetectPolicyChanged_.bind(this));
101 this.addWebUIListener( 102 this.addWebUIListener(
102 'can-set-date-time-changed', this.onCanSetDateTimeChanged_.bind(this)); 103 'can-set-date-time-changed', this.onCanSetDateTimeChanged_.bind(this));
103 104
104 chrome.send('dateTimePageReady'); 105 chrome.send('dateTimePageReady');
105 this.maybeGetTimeZoneList_(); 106 this.maybeGetTimeZoneList_();
106 }, 107 },
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
197 */ 198 */
198 setTimeZoneList_: function(timeZones) { 199 setTimeZoneList_: function(timeZones) {
199 this.timeZoneList_ = timeZones.map(function(timeZonePair) { 200 this.timeZoneList_ = timeZones.map(function(timeZonePair) {
200 return { 201 return {
201 name: timeZonePair[1], 202 name: timeZonePair[1],
202 value: timeZonePair[0], 203 value: timeZonePair[0],
203 }; 204 };
204 }); 205 });
205 }, 206 },
206 }); 207 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698