bluetooth: Add Device connection logic and accompanying user interface.

chrome/browser/resources/bluetooth_internals/bluetooth_internals.js

Index: chrome/browser/resources/bluetooth_internals/bluetooth_internals.js
diff --git a/chrome/browser/resources/bluetooth_internals/bluetooth_internals.js b/chrome/browser/resources/bluetooth_internals/bluetooth_internals.js
index 4057293273bc1ddaa7726553896bef63b21fd86d..c0439651fe05640cfb8ef24a368e64b465c19bec 100644
--- a/chrome/browser/resources/bluetooth_internals/bluetooth_internals.js
+++ b/chrome/browser/resources/bluetooth_internals/bluetooth_internals.js
@@ -8,6 +8,10 @@
  */
 
 cr.define('bluetooth_internals', function() {
+
+  /** @type {!Map<string, !interfaces.BluetoothDevice.Device.proxyClass>} */
+  var deviceAddressToProxy = new Map();
+
   function initializeViews() {
     var adapterBroker = null;
     adapter_broker.getAdapterBroker()
@@ -32,6 +36,49 @@ cr.define('bluetooth_internals', function() {
                                  devices /* this */);
 
         var deviceTable = new device_table.DeviceTable();
+
+        deviceTable.addEventListener('inspectpressed', function(event) {
+          // TODO(crbug.com/663470): Move connection logic to DeviceDetailsView
+          // when it's added in chrome://bluetooth-internals.
+          var address = event.detail.address;
+          var proxy = deviceAddressToProxy.get(address);
+
+          if (proxy) {
+            // Device is already connected, so disconnect.
+            proxy.disconnect();
+            deviceAddressToProxy.delete(address);
+            devices.updateConnectionStatus(
+                address, device_collection.ConnectionStatus.DISCONNECTED);
+            return;
+          }
+
+          devices.updateConnectionStatus(
+              address, device_collection.ConnectionStatus.CONNECTING);
+          adapterBroker.connectToDevice(address).then(function(deviceProxy) {

[dpapad, 2016/11/10 22:33:20]

Can you chain instead of nest Promise callbacks? Chained callbacks are less
readable, but also your current catch() call at line 74 will only catch some
errors but not all.


  return adapterBroker.connectToDevice(address);
}).then(function(deviceProxy) {
...
...
  return deviceProxy.getServices();
}).then(response) {
...
}).catch(e) {
  // catches any error that might have been generated above.
})

[dpapad, 2016/11/10 22:33:59]

Correction:
*nested callbacks are less readable

[mbrunson, 2016/11/11 21:26:14]

Done.

+            if (!devices.getByAddress(address)) {
+              // Device no longer in list, so drop the connection.
+              deviceProxy.disconnect();
+              return;
+            }
+
+            deviceAddressToProxy.set(address, deviceProxy);
+            devices.updateConnectionStatus(
+                address, device_collection.ConnectionStatus.CONNECTED);
+
+            // Fetch services asynchronously.
+            deviceProxy.getServices().then(function(response) {
+              var deviceInfo = devices.getByAddress(address);
+              deviceInfo.services = response.services;
+              devices.addOrUpdate(deviceInfo);
+            });
+          }).catch(function(error) {
+            devices.updateConnectionStatus(
+                address,
+                device_collection.ConnectionStatus.DISCONNECTED,
+                error);
+          });
+        });
+
         deviceTable.setDevices(devices);
         document.body.appendChild(deviceTable);
       })