HID fix for device info check.

chrome/browser/resources/chromeos/login/oobe_screen_hid_detection.js

 // Copyright 2014 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 Oobe HID detection screen implementation.
  */
 
 login.createScreen('HIDDetectionScreen', 'hid-detection', function() {
   return {
@@ skipping 11 matching lines @@
    PAIRING: {
      STARTUP: 'bluetoothStartConnecting',
      REMOTE_PIN_CODE: 'bluetoothRemotePinCode',
      REMOTE_PASSKEY: 'bluetoothRemotePasskey',
      CONNECT_FAILED: 'bluetoothConnectFailed',
      CANCELED: 'bluetoothPairingCanceled',
      // Pairing dismissed (succeeded or canceled).
      DISMISSED: 'bluetoothPairingDismissed'
    },
 
+   // Enumeration of possible connection states of a device.
+   CONNECTION: {
+     SEARCHING: 'searching',
+     CONNECTED: 'connected',
+     PAIRING: 'pairing',
+     PAIRED: 'paired',
+     // Special info state.
+     UPDATE: 'update'
+   },
+
+   // Possible ids of device blocks.
+   BLOCK: {
+     MOUSE: 'hid-mouse-block',
+     KEYBOARD: 'hid-keyboard-block'
+   },
+
     /**
      * Button to move to usual OOBE flow after detection.
      * @private
      */
     continueButton_: null,
 
     /**
      * Buttons in oobe wizard's button strip.
      * @type {array} Array of Buttons.
      */
@@ skipping 16 matching lines @@
     /**
      * Returns a control which should receive an initial focus.
      */
     get defaultControl() {
       return this.continueButton_;
     },
 
     /**
      * Sets a device-block css class to reflect device state of searching,
      * connected, pairing or paired (for BT devices).
-     * @param {blockId} id one of 'hid-mouse-block' or 'hid-keyboard-block'.
-     * @param {state} one of 'searching', 'connected', 'pairing', 'paired',
+     * @param {blockId} id one of keys of this.BLOCK dict.
+     * @param {state} one of keys of this.CONNECTION dict.
      * @private
      */
     setDeviceBlockState_: function(blockId, state) {
       if (state == 'update')
         return;
       var deviceBlock = $(blockId);
-      var states = ['searching', 'connected', 'pairing', 'paired'];
-      for (var i = 0; i < states.length; ++i) {
-        if (states[i] != state)
-          deviceBlock.classList.remove(states[i]);
+      for (var stateCase in this.CONNECTION) {
+        if (stateCase != state)
+          deviceBlock.classList.remove(stateCase);

[Denis Kuznetsov (DE-MUC), 2015/02/05 13:26:17]

You can use .toggle(name, value) here.

       }
       deviceBlock.classList.add(state);
+
+      // 'Continue' button available iff at least one device is connected,
+      if ((blockId in this.BLOCK) &&
+          (state == this.CONNECTION.CONNECTED ||
+           state == this.CONNECTION.PAIRED)) {
+        $('hid-continue-button').disabled = false;
+      } else {
+        $('hid-continue-button').disabled = true;
+      }
     },
 
     /**
      * Sets state for mouse-block.
-     * @param {state} one of 'searching', 'connected', 'paired'.
+     * @param {state} one of keys of this.CONNECTION dict.
      */
     setPointingDeviceState: function(state) {
       if (state === undefined)
         return;
-      this.setDeviceBlockState_('hid-mouse-block', state);
+      this.setDeviceBlockState_(this.BLOCK.MOUSE, state);
     },
 
     /**
      * Sets state for pincode key elements.
      * @param {entered} int, number of typed keys of pincode, -1 if keys press
      * detection is not supported by device.
      */
     setPincodeKeysState_: function(entered) {
       var pincodeLength = 7; // including enter-key
       for (var i = 0; i < pincodeLength; i++) {
@@ skipping 11 matching lines @@
     },
 
     /**
      * Sets state for keyboard-block.
      * @param {data} dict with parameters.
      */
     setKeyboardDeviceState: function(data) {
       if (data === undefined || !('state' in data))
         return;
       var state = data['state'];
-      this.setDeviceBlockState_('hid-keyboard-block', state);
-      if (state == 'paired')
+      this.setDeviceBlockState_(this.BLOCK.KEYBOARD, state);
+      if (state == this.CONNECTION.PAIRED)
         $('hid-keyboard-label-paired').textContent = data['keyboard-label'];
-      else if (state == 'pairing') {
+      else if (state == this.CONNECTION.PAIRING) {
         $('hid-keyboard-label-pairing').textContent = data['keyboard-label'];
         if (data['pairing-state'] == this.PAIRING.REMOTE_PIN_CODE ||
             data['pairing-state'] == this.PAIRING.REMOTE_PASSKEY) {
           this.setPincodeKeysState_(-1);
           for (var i = 0, len = data['pincode'].length; i < len; i++) {
             var pincodeSymbol = $('hid-keyboard-pincode-sym-' + (i + 1));
             pincodeSymbol.textContent = data['pincode'][i];
           }
           announceAccessibleMessage(
               data['keyboard-label'] + ' ' + data['pincode'] + ' ' +
               loadTimeData.getString('hidDetectionBTEnterKey'));
         }
-      } else if (state == 'update') {
+      } else if (state == this.CONNECTION.UPDATE) {
         if ('keysEntered' in data) {
           this.setPincodeKeysState_(data['keysEntered']);
         }
       }
     },
-
-    /**
-     * Event handler that is invoked just before the screen in shown.
-     * @param {Object} data Screen init payload.
-     */
-    onBeforeShow: function(data) {
-      $('hid-continue-button').disabled = true;
-      this.setDeviceBlockState_('hid-mouse-block', 'searching');
-      this.setDeviceBlockState_('hid-keyboard-block', 'searching');
-    },
   };
 });