bluetooth: Add Device connection logic and accompanying user interface.

chrome/browser/resources/bluetooth_internals/device_table.js

 // Copyright 2016 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.
 
 /**
  * Javascript for DeviceTable UI, served from
  *     chrome://bluetooth-internals/.
  */
 
 cr.define('device_table', function() {
@@ skipping 29 matching lines @@
      * Sets the tables device collection.
      * @param {!device_collection.DeviceCollection} deviceCollection
      */
     setDevices: function(deviceCollection) {
       assert(!this.devices_, 'Devices can only be set once.');
 
       this.devices_ = deviceCollection;
       this.devices_.addEventListener('sorted', this.redraw_.bind(this));
       this.devices_.addEventListener('change', this.handleChange_.bind(this));
       this.devices_.addEventListener('splice', this.handleSplice_.bind(this));
+      this.devices_.addEventListener(
+          'connectstatus', this.handleStatus_.bind(this));
 
       this.redraw_();
     },
 
     /**
      * Updates table row on change event of the device collection.
      * @private
      * @param {!CustomEvent} event
      */
     handleChange_: function(event) {
       this.updateRow_(this.devices_.item(event.index), event.index);
     },
 
     /**
+     * Fires a connect request event for the row |index|.
+     * @private
+     * @param {number} index
+     */
+    handleConnectBtn_: function(index) {
+      var event = new CustomEvent('connectrequest', {
+        detail: {
+          index: index,

[ortuno, 2016/11/09 03:24:03]

I would include the address rather than the index since the index can change
during the connection.

[mbrunson, 2016/11/09 23:39:37]

Done.

+        }
+      });
+      bluetooth_internals.publish(event);
+    },
+
+    /**
      * Updates table row on splice event of the device collection.
      * @private
      * @param {!CustomEvent} event
      */
     handleSplice_: function(event) {
       event.removed.forEach(function() {
         this.body_.deleteRow(event.index);
       }, this);
 
       event.added.forEach(function(device, index) {
         this.insertRow_(device, event.index + index);
       }, this);
     },
 
+    handleStatus_: function(event) {
+      var index = event.detail.index;
+      this.updateRow_(this.devices_.item(index), index);
+
+      var row = this.body_.rows[index];
+      var cellCount = row.cells.length;
+      var connectErrorCell = row.cells[cellCount - 1];
+      connectErrorCell.textContent = event.detail.message;
+    },
+
     /**
      * Inserts a new row at |index| and updates it with info from |device|.
      * @private
      * @param {!device_collection.Device} device
      * @param {?number} index
      */
     insertRow_: function(device, index) {
       var row = this.body_.insertRow(index);
       row.id = device.info.address;
 
       for (var i = 0; i < this.headers_.length; i++) {
         row.insertCell();
       }
 
+      // Make two extra cells for the connect button and connect errors.
+      var connectCell = row.insertCell();
+      var connectErrorCell = row.insertCell();

[ortuno, 2016/11/09 03:24:03]

Would it make sense to show a snackbar[1] instead? No need to do it in this
patch but if you decide to go in that direction you can just leave a TODO.

[1] https://material.google.com/components/snackbars-toasts.html#

[mbrunson, 2016/11/09 23:39:37]

Yeah. That would be ideal. TODO added.

+
+      var connectButton = document.createElement('button');
+      connectCell.appendChild(connectButton);
+      connectButton.addEventListener('click', function() {
+        this.handleConnectBtn_(row.sectionRowIndex);
+      }.bind(this));
+
       this.updateRow_(device, row.sectionRowIndex);
     },
 
     /**
      * Deletes and recreates the table using the cached |devices_|.
      * @private
      */
     redraw_: function() {
       this.removeChild(this.body_);
       this.appendChild(document.createElement('tbody'));
@@ skipping 14 matching lines @@
     updateRow_: function(device, index) {
       assert(this.body_.rows[index], 'Row ' + index + ' is not in the table.');
       var row = this.body_.rows[index];
 
       if (device.removed) {
         row.classList.add(REMOVED_CSS);
       } else {
         row.classList.remove(REMOVED_CSS);
       }
 
+      var cellCount = row.cells.length;
+      var connectButton = row.cells[cellCount - 2].children[0];
+      if (device.proxy) {

[ortuno, 2016/11/09 03:24:03]

switch (device.connectionStatus) {
  case 'disconnected':
    connectButton.textContent = 'Forget';
    break;
  case 'connected':
    connectButton.textContent = 'Inspect';
    break;
  case 'connecting':
    // disable button.
    break;
  default: assert('case not handled');
}

[mbrunson, 2016/11/09 23:39:37]

Done.

+        connectButton.textContent = 'Forget';
+      } else {
+        connectButton.textContent = 'Inspect';
+      }
+
       // Update the properties based on the header field path.
       for (var i = 0; i < this.headers_.length; i++) {
         var header = this.headers_[i];
         var propName = header.dataset.field;
 
         var parts = propName.split('.');
         var obj = device.info;
         while (obj != null && parts.length > 0) {
           var part = parts.shift();
           obj = obj[part];
         }
 
+        if (propName == 'is_gatt_connected') {
+          obj ? obj = 'Connected' : obj = 'Not Connected';
+        }
+
         var cell = row.cells[i];
-        cell.textContent = obj || 'Unknown';
+        cell.textContent = (obj == null || obj == undefined) ? 'Unknown' : obj;
         cell.dataset.label = header.textContent;
       }
     },
   };
 
   return {
     DeviceTable: DeviceTable,
   };
 });