MD Settings: Add mouse settings, update pointer settings

chrome/browser/resources/settings/device_page/device_page.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.
 
 /**
  * @fileoverview 'settings-device-page' is the settings page for device and
  * peripheral settings.
  */
 Polymer({
   is: 'settings-device-page',
 
+  behaviors: [
+    I18nBehavior,
+    WebUIListenerBehavior,
+  ],
+
   properties: {
     /** The current active route. */
     currentRoute: {
       type: Object,
       notify: true,
     },
 
     /** Preferences state. */
     prefs: {
       type: Object,
       notify: true,
     },
+
+    /** @private */
+    hasMouse_: {
+      type: Boolean,
+      value: false,
+    },
+
+    /** @private */
+    hasTouchpad_: {
+      type: Boolean,
+      value: false,
+    },
+  },
+
+  ready: function() {
+    cr.addWebUIListener(
+        'has-mouse-changed', this.hasMouseChanged_.bind(this));
+    cr.addWebUIListener(
+        'has-touchpad-changed', this.hasTouchpadChanged_.bind(this));
+    settings.DevicePageBrowserProxyImpl.getInstance().initializePointers();
   },
 
   /**
-   * Handler for tapping the Touchpad settings menu item.
+   * Callback to set whether a mouse is connected.
+   * @param {boolean} hasMouse
    * @private
    */
-  onTouchpadTap_: function() {
-    this.$.pages.setSubpageChain(['touchpad']);
+  hasMouseChanged_: function(hasMouse) {
+    this.hasMouse_ = hasMouse;
   },
 
   /**
+   * Callback to set whether a touchpad is connected.
+   * @param {boolean} hasTouchpad
+   * @private
+   */
+  hasTouchpadChanged_: function(hasTouchpad) {
+    this.hasTouchpad_ = hasTouchpad;
+  },

[stevenjb, 2016/06/30 00:07:49]

nit: Any reason not to just inline these two fairly trivial functions?

[michaelpg, 2016/07/09 04:11:30]

Done.

+
+  /**
+   * @return {string}
+   * @private
+   */
+  getPointersTitle_: function() {
+    if (this.hasMouse_ && this.hasTouchpad_)
+      return this.i18n('mouseAndTouchpadTitle');
+    if (this.hasMouse_)
+      return this.i18n('mouseTitle');
+    if (this.hasTouchpad_)
+      return this.i18n('touchpadTitle');
+    return '';
+  },
+
+  /**
+   * @return {string}
+   * @private
+   */
+  getPointersIcon_: function() {
+    if (this.hasMouse_)
+      return 'settings:mouse';
+    if (this.hasTouchpad_)
+      return 'settings:touch-app';
+    return '';
+  },
+
+  /**
+   * Handler for tapping the mouse and touchpad settings menu item.
+   * @private
+   */
+  onPointersTap_: function() {
+    this.$.pages.setSubpageChain(['pointers']);
+  },
+
+  /**
    * Handler for tapping the Keyboard settings menu item.
    * @private
    */
   onKeyboardTap_: function() {
     this.$.pages.setSubpageChain(['keyboard']);
   },
 
   /**
    * Handler for tapping the Display settings menu item.
    * @private
    */
   onDisplayTap_: function() {
     this.$.pages.setSubpageChain(['display']);
   },
+
+  /**
+   * @return {boolean}
+   * @private
+   */
+  showPointers_: function(hasMouse, hasTouchpad) {
+    return hasMouse || hasTouchpad;
+  },
 });