MD Settings: adjust iron-list focus row behaviors.

chrome/browser/resources/settings/focus_row_behavior.js

Index: chrome/browser/resources/settings/focus_row_behavior.js
diff --git a/chrome/browser/resources/settings/focus_row_behavior.js b/chrome/browser/resources/settings/focus_row_behavior.js
new file mode 100644
index 0000000000000000000000000000000000000000..c956e2d8380be6311fd85f49a0a6670e6ea5b0b0
--- /dev/null
+++ b/chrome/browser/resources/settings/focus_row_behavior.js
@@ -0,0 +1,172 @@
+// Copyright 2017 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.
+
+/**
+ * @param {{lastFocused: Object}} listItem
+ * @constructor
+ * @implements {cr.ui.FocusRow.Delegate}
+ */
+function FocusRowDelegate(listItem) {
+  /** @private */
+  this.listItem_ = listItem;
+}
+
+FocusRowDelegate.prototype = {
+  /**
+   * @override
+   * @param {!cr.ui.FocusRow} row
+   * @param {!Event} e
+   */
+  onFocus: function(row, e) {
+    this.listItem_.lastFocused = e.path[0];
+  },
+
+  /**
+   * @override
+   * @param {!cr.ui.FocusRow} row The row that detected a keydown.
+   * @param {!Event} e
+   * @return {boolean} Whether the event was handled.
+   */
+  onKeydown: function(row, e) {
+    // Prevent iron-list from changing the focus on enter.
+    if (e.key == 'Enter')
+      e.stopPropagation();
+
+    return false;
+  },
+};
+
+/**
+ * @param {!Element} root
+ * @param {cr.ui.FocusRow.Delegate} delegate
+ * @constructor
+ * @extends {cr.ui.FocusRow}
+ */
+function VirtualFocusRow(root, delegate) {
+  cr.ui.FocusRow.call(this, root, /* boundary */ null, delegate);
+}
+
+VirtualFocusRow.prototype = {
+  __proto__: cr.ui.FocusRow.prototype
+};
+
+
+/**
+ * Any element that is being used as an iron-list row item can extend this
+ * behavior, which encapsulates focus controls of mouse and keyboards.
+ * To use this behavior:
+ *    - The parent element should pass a "last-focused" attribute double-bound
+ *      to the row items, to track the last-focused element across rows.
+ *    - There must be a container in the extending element with the
+ *      [focus-row-container] attribute that contains all focusable controls.
+ *    - On each of the focusable controls, there must be a [focus-row-control]
+ *      attribute, and a [type=] attribute unique for each type of control.
+ *
+ * @polymerBehavior
+ */
+var FocusRowBehavior = {
+  properties: {
+    /** @private {VirtualFocusRow} */
+    row_: Object,
+
+    /** @private {boolean} */
+    mouseFocused_: Boolean,
+
+    /** @type {Element} */
+    lastFocused: {
+      type: Object,
+      notify: true,
+    },
+
+    /**
+     * This is different from tabIndex, since the template only does a one-way
+     * binding on both attributes, and the behavior actually make use of this
+     * fact. For example, when a control within a row is focused, it will have
+     * tabIndex = -1 and ironListTabIndex = 0.
+     * @type {number}
+     */
+    ironListTabIndex: {
+      type: Number,
+      observer: 'ironListTabIndexChanged_',
+    },
+  },
+
+  /** @override */
+  attached: function() {
+    this.classList.add('no-outline');
+
+    Polymer.RenderStatus.afterNextRender(this, function() {
+      var rowContainer = this.root.querySelector('[focus-row-container]');
+      assert(!!rowContainer);
+      this.row_ = new VirtualFocusRow(rowContainer, new FocusRowDelegate(this));
+      this.ironListTabIndexChanged_();
+      this.addItems_();
+
+      // Adding listeners asynchronously to reduce blocking time, since this
+      // behavior will be used by items in potentially long lists.
+      this.listen(this, 'focus', 'onFocus_');
+      this.listen(this, 'dom-change', 'addItems_');
+      this.listen(this, 'mousedown', 'onMouseDown_');
+      this.listen(this, 'blur', 'onBlur_');
+    });
+  },
+
+  /** @override */
+  detached: function() {
+    this.unlisten(this, 'focus', 'onFocus_');
+    this.unlisten(this, 'dom-change', 'addItems_');
+    this.unlisten(this, 'mousedown', 'onMouseDown_');
+    this.unlisten(this, 'blur', 'onBlur_');
+    if (this.row_)
+      this.row_.destroy();
+  },
+
+  /** @private */
+  addItems_: function() {
+    if (this.row_) {
+      this.row_.destroy();
+
+      var controls = this.root.querySelectorAll('[focus-row-control]');
+
+      for (var i = 0; i < controls.length; i++) {
+        this.row_.addItem(
+            controls[i].getAttribute('type'),

[Dan Beam, 2017/03/21 07:11:52]

can we make this focus-type instead of just type?

[scottchen, 2017/03/21 20:10:33]

Done.

+            /** @type {HTMLElement} */ (controls[i]));
+      }
+    }
+  },
+
+  /** @private */
+  onFocus_: function() {
+    if (this.mouseFocused_) {
+      this.mouseFocused_ = false;  // Consume and reset flag.
+      return;
+    }
+
+    if (this.lastFocused) {
+      this.row_.getEquivalentElement(this.lastFocused).focus();
+    } else {
+      var firstFocusable = assert(this.row_.getFirstFocusable());
+      firstFocusable.focus();
+    }
+
+    this.tabIndex = -1;
+  },
+
+  /** @private */
+  ironListTabIndexChanged_: function() {
+    if (this.row_)
+      this.row_.makeActive(this.ironListTabIndex == 0);
+  },
+
+  /** @private */
+  onMouseDown_: function() {
+    this.mouseFocused_ = true;  // Set flag to not do any control-focusing.
+  },
+
+  /** @private */
+  onBlur_: function() {
+    this.mouseFocused_ = false;  // Reset flag since it's not active anymore.
+  }
+};