[MD settings] end of page padding in content area

chrome/browser/resources/settings/settings_main/settings_main.js

 // Copyright 2015 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-main' displays the selected settings page.
  */
 Polymer({
   is: 'settings-main',
@@ skipping 120 matching lines @@
 
     this.showAdvancedToggle_ = !this.showAboutPage_ && !isSubpage;
 
     this.showBasicPage_ = this.showAdvancedToggle_ || newRoute.page == 'basic';
 
     this.showAdvancedPage_ =
         (this.isAdvancedMenuOpen_ && this.showAdvancedToggle_) ||
         newRoute.page == 'advanced';
 
     this.style.height = isSubpage ? '100%' : '';
+
+    // Wait for the dom-if changes prior to calculating the overflow padding.
+    this.async(function() {
+      this.$.overscroll.style.paddingBottom = this.overscrollHeight_() + 'px';
+    });
+  },
+
+  /**
+   * Return the height that the over scroll padding should be set to.
+   * This is used to determine how much padding to apply to the end of the
+   * content so that the last element may align with the top of the content
+   * area.
+   * @return {number}
+   * @private
+   */
+  overscrollHeight_: function() {
+    if (!this.currentRoute || this.currentRoute.subpage.length != 0 ||
+        this.showAboutPage_) {
+      return 0;
+    }
+
+    /**
+     * @param {!Element} element
+     * @return {number}
+     */
+    function calcHeight(scrollHeight, element) {

[Dan Beam, 2016/07/20 21:22:39]

nit: missing a @param, BUT, can we just get this value when we need it?

var calcHeight = function(element) {
  var height = this.parentNode.scrollHeight - ...;
}.bind(this);

if we do end up needing a variable, should the name "availableHeight" be more
explanatory?

[dschuyler, 2016/07/21 00:14:23]

That is nicer, thanks.

Done.

+      var style = getComputedStyle(element);
+      var height = scrollHeight - element.offsetHeight +
+          parseFloat(style.marginTop) + parseFloat(style.marginBottom);
+      assert(height >= 0);
+      return height;
+    }
+
+    var scrollHeight = this.parentNode.scrollHeight;
+
+    if (this.showAdvancedPage_) {
+      var lastSection = this.$$('settings-advanced-page').$$(
+          'settings-section:last-of-type');
+      // |lastSection| may be null in unit tests.
+      if (lastSection)
+        return calcHeight(scrollHeight, lastSection);
+    }
+
+    var toggleContainer = this.$$('#toggleContainer');
+    if (toggleContainer)
+      return calcHeight(scrollHeight, toggleContainer);
+
+    return 0;
   },
 
   /** @private */
   toggleAdvancedPage_: function() {
     this.fire('toggle-advanced-page', !this.isAdvancedMenuOpen_);
   },
 
   /**
    * Navigates to the default search page (if necessary).
    * @private
@@ skipping 21 matching lines @@
           query, assert(this.$$('settings-basic-page')));
     }.bind(this), 0);
 
     this.showAdvancedPage_ = true;
     setTimeout(function() {
       settings.getSearchManager().search(
           query, assert(this.$$('settings-advanced-page')));
     }.bind(this), 0);
   },
 });