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

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

Issue 2651483002: MD Settings: Eliminate use of ES6 for Chrome OS (Closed)
Patch Set: . Created 3 years, 11 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-bluetooth-page' is the settings page for managing bluetooth 7 * 'settings-bluetooth-page' is the settings page for managing bluetooth
8 * properties and devices. 8 * properties and devices.
9 * 9 *
10 * Example: 10 * Example:
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 221
222 /** @private */ 222 /** @private */
223 bluetoothEnabledChanged_: function() { 223 bluetoothEnabledChanged_: function() {
224 // When bluetooth is enabled, auto-expand the device list. 224 // When bluetooth is enabled, auto-expand the device list.
225 if (this.bluetoothEnabled_) 225 if (this.bluetoothEnabled_)
226 this.deviceListExpanded_ = true; 226 this.deviceListExpanded_ = true;
227 }, 227 },
228 228
229 /** @private */ 229 /** @private */
230 deviceListChanged_: function() { 230 deviceListChanged_: function() {
231 for (let device of this.deviceList_) { 231 for (var d = 0; d <this.deviceList_.length; ++d) {
dpapad 2017/01/23 19:12:53 Space missing before |this.deviceList_.length|. Al
stevenjb 2017/01/24 02:02:03 done / ack. I tend to prefer more context relevent
232 if (device.connected) { 232 if (this.deviceList_[d].connected) {
233 this.deviceConnected_ = true; 233 this.deviceConnected_ = true;
234 return; 234 return;
235 } 235 }
236 } 236 }
237 this.deviceConnected_ = false; 237 this.deviceConnected_ = false;
238 }, 238 },
239 239
240 /** @private */ 240 /** @private */
241 selectedItemChanged_: function() { 241 selectedItemChanged_: function() {
242 if (this.selectedItem_) 242 if (this.selectedItem_)
(...skipping 375 matching lines...) Expand 10 before | Expand all | Expand 10 after
618 }, 618 },
619 619
620 /** 620 /**
621 * @param {Event} e 621 * @param {Event} e
622 * @private 622 * @private
623 */ 623 */
624 stopTap_: function(e) { 624 stopTap_: function(e) {
625 e.stopPropagation(); 625 e.stopPropagation();
626 }, 626 },
627 }); 627 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698