MD Settings: polish top-level bluetooth page

chrome/browser/resources/settings/bluetooth_page/bluetooth_page.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview
  * 'settings-bluetooth-page' is the settings page for managing bluetooth
  *  properties and devices.
  *
  * Example:
@@ skipping 179 matching lines @@
     }
   },
 
   bluetoothEnabledChanged_: function() {
     // When bluetooth is enabled, auto-expand the device list.
     if (this.bluetoothEnabled)
       this.deviceListExpanded = true;
   },
 
   /**
+   * @return {boolean} Whether the <iron-collapse> can be shown.
+   * @private
+   */
+  canShowDeviceList_: function() {

[stevenjb, 2016/04/12 15:58:49]

We never resolved this in the style guide, but we should at least be consistent
within a file. The rest of this file declares and uses the passed in properties.

FWIW at first look I thought this might be a bug since it appears that the
properties were not being passed to this.

Also, I just fixed a Polymer bug where only one out of 4 dependent properties
was being passed to a function. It was easy to spot the bug since the function
was using this.bar but not passing bar to the function.

[Dan Beam, 2016/04/12 19:51:02]

Done.

+    return this.bluetoothEnabled && this.deviceListExpanded;
+  },
+
+  /**
    * If bluetooth is enabled, request the complete list of devices and update
    * |deviceList|.
    * @private
    */
   updateDeviceList_: function() {
     if (!this.bluetoothEnabled) {
       this.deviceList = [];
       return;
     }
     this.bluetooth.getDevices(function(devices) {
@@ skipping 301 matching lines @@
 
   /** @private */
   onCloseDialog_: function(event) { this.closeDialog_(); },
 
   /** @private */
   onDialogOpened_: function() { this.startDiscovery_(); },
 
   /** @private */
   onDialogClosed_: function() { this.stopDiscovery_(); },
 });