MD Settings: Introduce CrScrollableBehavior

ui/webui/resources/cr_elements/cr_scrollable_behavior.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 Behavior for scrollable containers with <iron-list>.
+ *
+ * Any containers with the 'scrollable' attribute set will have the following
+ * classes toggled appropriately: can-scroll, is-scrolled, scrolled-to-bottom.
+ * These classes are used to style the container div and list elements
+ * appropriately, see shared_style_css.html.
+ *
+ * The associated HTML should look something like:
+ *   <div id="container" scrollable>
+ *     <iron-list items="[[items]]" scroll-target="container">
+ *       <template>
+ *         <my-element item="[[item]] tabindex$="[[tabIndex]]"></my-element>
+ *       </template>
+ *     </iron-list>
+ *   </div>
+ *
+ * In order to get correct keyboard focus (tab) behavior within the list,
+ * any elements with tabbable sub-elements also need to set tabindex, e.g:
+ *
+ * <dom-module id="my-element>
+ *   <template>
+ *     ...
+ *     <paper-icon-button toggles active="{{opened}}" tabindex$="[[tabindex]]">
+ *   </template>
+ * </dom-module>
+ *
+ * NOTE: If 'container' is not fixed size, it is important to call
+ * updateScrollableContents() when [[items]] changes, otherwise the container
+ * will not be sized correctly.
+ */
+
+/** @polymerBehavior */
+var CrScrollableBehavior = {
+  attached: function() {
+    var scrollableElements = this.root.querySelectorAll('[scrollable]');
+
+    // Setup the intial scrolling related classes for each scrollable container.
+    requestAnimationFrame(function() {
+      for (let scrollable of scrollableElements)
+        this.updateScroll_(scrollable);
+    }.bind(this));
+
+    // Listen to the 'scroll' event for each scrollable container.
+    for (let scrollable of scrollableElements)
+      scrollable.addEventListener('scroll', this.updateScroll_);
+  },
+
+  detached: function() {
+    // Remove 'scroll' event listeners.
+    var scrollableElements = this.root.querySelectorAll('[scrollable]');
+    for (let scrollable of scrollableElements)
+      scrollable.removeEventListener('scroll', this.updateScroll_);
+  },
+
+  /**
+   * Called any time the contents of a scrollable container may have changed.
+   * This ensures that the <iron-list> contents of dynamically sized
+   * containers are resized correctly.
+   */
+  updateScrollableContents() {
+    let nodeList = this.root.querySelectorAll('[scrollable] iron-list');
+    // Use setTimeout to avoid initial render / sizing issues.

[dschuyler, 2016/08/18 01:25:11]

Would doWhenReady help?
https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/settin...

The doWhenReady is preferred because it is more robust and 
helps highlight all the areas that are suffering from this
issue (which Michaelpg@ has been looking for a solution to).
Here's an example use:

https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/settin...

+    setTimeout(function() {
+      for (let node of nodeList) {
+        let ironList = /** @type {!IronListElement} */ (node);
+        ironList.notifyResize();
+      }
+    });
+  },
+
+  /**
+   * This gets called once intially and any time a scrollable container scrolls.
+   * @param {Event} event
+   * @private
+   */
+  updateScroll_(event) {
+    let scrollable = event.target;
+    scrollable.classList.toggle(
+        'can-scroll', scrollable.clientHeight < scrollable.scrollHeight);
+    scrollable.classList.toggle('is-scrolled', scrollable.scrollTop > 0);
+    scrollable.classList.toggle(
+        'scrolled-to-bottom', scrollable.scrollTop + scrollable.clientHeight >=
+            scrollable.scrollHeight);
+  },
+};