Allow navigation to chrome://md-settings/pointer-overlay

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,
     settings.RouteObserverBehavior,
   ],
 
   properties: {
-    /** Preferences state. */
     prefs: {
       type: Object,
       notify: true,
     },
 
-    /** @private */
-    hasMouse_: {
-      type: Boolean,
-      value: false,
-    },
+    /**
+     * |hasMouse_| and |hasTouchpad_| start undefined so observers don't trigger
+     * until both have been populated.
+     * @private
+     */
+    hasMouse_: Boolean,
 
     /** @private */
-    hasTouchpad_: {
-      type: Boolean,
-      value: false,
-    },
+    hasTouchpad_: Boolean,
 
     /** @private */
     stylusAllowed_: {
       type: Boolean,
       value: function() {
         return loadTimeData.getBoolean('stylusAllowed');
       },
       readOnly: true,
     },
   },
 
   observers: [
     'pointersChanged_(hasMouse_, hasTouchpad_)',
   ],
 
+  /** @override */
   ready: function() {
-    cr.addWebUIListener('has-mouse-changed', this.set.bind(this, 'hasMouse_'));
-    cr.addWebUIListener(
+    settings.DevicePageBrowserProxyImpl.getInstance().initializePointers();
+  },
+
+  /** @override */
+  attached: function() {
+    this.addWebUIListener(
+        'has-mouse-changed', this.set.bind(this, 'hasMouse_'));
+    this.addWebUIListener(
         'has-touchpad-changed', this.set.bind(this, 'hasTouchpad_'));
-    settings.DevicePageBrowserProxyImpl.getInstance().initializePointers();
   },
 
   /**
    * @return {string}
    * @private
    */
   getPointersTitle_: function() {
     if (this.hasMouse_ && this.hasTouchpad_)
       return this.i18n('mouseAndTouchpadTitle');
     if (this.hasMouse_)
@@ skipping 68 matching lines @@
   pointersChanged_: function(hasMouse, hasTouchpad) {
     this.$.pointersRow.hidden = !hasMouse && !hasTouchpad;
     this.checkPointerSubpage_();
   },
 
   /**
    * Leaves the pointer subpage if all pointing devices are detached.
    * @private
    */
   checkPointerSubpage_: function() {
-    if (!this.hasMouse_ && !this.hasTouchpad_ &&
+    // Check that the properties have explicitly been set to false.
+    if (this.hasMouse_ === false && this.hasTouchpad_ === false &&
         settings.getCurrentRoute() == settings.Route.POINTERS) {
       settings.navigateTo(settings.Route.DEVICE);
     }
   },
 });