Add display of available bluetooth devices, and mechanism for retrieving the list.

chrome/browser/resources/options/chromeos/system_options.js

 // Copyright (c) 2011 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.
 
 cr.define('options', function() {
 
   var OptionsPage = options.OptionsPage;
   /////////////////////////////////////////////////////////////////////////////
   // SystemOptions class:
 
@@ skipping 23 matching lines @@
       // Disable time-related settings if we're not logged in as a real user.
       if (AccountsOptions.loggedInAsGuest()) {
         var timezone = $('timezone-select');
         if (timezone)
           timezone.disabled = true;
         var use_24hour_clock = $('use-24hour-clock');
         if (use_24hour_clock)
           use_24hour_clock.disabled = true;
       }
 
+      options.system.bluetooth.BluetoothListElement.decorate(
+          $('bluetooth-device-list'));
+
       // TODO (kevers) - Populate list of connected bluetooth devices.
       //                 Set state of 'Enable bluetooth' checkbox.
-      //                 Set state of 'Find devices' button.
+
+      $('bluetooth-find-devices').disabled = $('enable-bluetooth-label') ?

[James Hawkins, 2011/10/04 20:43:39]

Wrap after '='.

[kevers, 2011/10/05 14:23:01]

Done.

+          false : true;
+      $('bluetooth-find-devices').onclick = function(event) {
+        findBluetoothDevices_();
+      };
 
       $('language-button').onclick = function(event) {
         OptionsPage.navigateToPage('language');
       };
       $('modifier-keys-button').onclick = function(event) {
         OptionsPage.navigateToPage('languageCustomizeModifierKeysOverlay');
       };
       $('accesibility-check').onchange = function(event) {
         chrome.send('accessibilityChange',
                     [String($('accesibility-check').checked)]);
@@ skipping 12 matching lines @@
     $('accesibility-check').checked = checked;
   };
 
   /**
    * Activate the bluetooth settings section on the System settings page.
    */
   SystemOptions.ShowBluetoothSettings = function() {
     $('bluetooth-devices').hidden = false;
   }
 
+  /**
+   * Adds an element to the list of available bluetooth devices.
+   * @param{{'deviceName': string,
+   *         'deviceId': string,
+   *         'deviceType': Constants.DEVICE_TYPE,
+   *         'deviceStatus': Constants.DEVICE_STATUS} device
+   *     Decription of the bluetooth device.
+   */
+  SystemOptions.AddBluetoothDevice = function(device) {
+    $('bluetooth-device-list').appendDevice(device);
+  }
+
+  /**
+   * Hides the scanning label and icon that are used to indicate that a device
+   * search is in progress.
+   */
+  SystemOptions.NotifyBluetoothSearchComplete = function() {
+    // TODO (kevers) - reset state immediately once fake response is made
+    //                 asynchronously.
+    setTimeout( function() {
+      $('bluetooth-scanning-label').hidden = true;
+      $('bluetooth-scanning-icon').hidden = true;

[James Hawkins, 2011/10/04 20:43:39]

Are these hides animated?

[kevers, 2011/10/05 14:23:01]

No, the transition is not animated.  Is there a suggested transition duration?

[kevers, 2011/10/05 20:32:39]

The call to setTimeout is a temporary visual hack (for story-boarding) that will
be removed once the response is asynchronous.

[James Hawkins, 2011/10/05 20:35:28]

Yes, hides should be animated...I'm not sure about the suggested transition
duration. You can check the css files in resources for examples.

[kevers, 2011/10/06 14:51:25]

The hide now has a 0.25s fade animation.

+    }, 2000);
+  }
+
+  /**
+   * Scan for bluetooth devices.
+   * @private
+   */
+  function findBluetoothDevices_() {
+    console.log('Scanning for bluetooth devices.');

[James Hawkins, 2011/10/04 20:43:39]

Remove console spam.

[kevers, 2011/10/05 14:23:01]

Oops.  My bad.

+    $('bluetooth-scanning-label').hidden = false;
+    $('bluetooth-scanning-icon').hidden = false;
+
+    // Remove devices that are not currently connected.
+    var devices = $('bluetooth-device-list').childNodes;
+    for (var i = devices.length - 1; i >= 0; i--) {
+      var device = devices.item(i);
+      var data = device.data;
+      if (!data || data.status !== 'connected')
+        $('bluetooth-device-list').removeChild(device);
+    }
+    // TODO (kevers) - Set arguments to a list of the currently connected
+    //                 devices.
+    chrome.send('findBluetoothDevices', []);

[James Hawkins, 2011/10/04 20:43:39]

No need for final (empty) parameter.

[kevers, 2011/10/05 14:23:01]

Done.

+  }
+
   // Export
   return {
     SystemOptions: SystemOptions
   };
 
 });