Merge branch 'master' into 4458-kschaaf-basic-templatizer

.github/ISSUE_TEMPLATE.md

@@ -1,6 +1,6 @@
 <!--
 If you are asking a question rather than filing a bug, try one of these instead:
-- StackOverflow (http://stackoverflow.com/questions/tagged/polymer)
+- StackOverflow (https://stackoverflow.com/questions/tagged/polymer)
 - Polymer Slack Channel (https://bit.ly/polymerslack)
 - Mailing List (https://groups.google.com/forum/#!forum/polymer-dev)
 -->
@@ -10,7 +10,7 @@ If you are asking a question rather than filing a bug, try one of these instead:
 
 #### Live Demo
 <!-- Fork this JSBin, or provide your own URL -->
-http://jsbin.com/luhaxab/1/edit
+https://jsbin.com/luhaxab/1/edit
 
 #### Steps to Reproduce
 <!--

.travis.yml

@@ -1,5 +1,5 @@
 language: node_js
-sudo: false
+sudo: required
 dist: trusty
 node_js: stable
 addons:
@@ -15,9 +15,8 @@ before_script:
 - gulp update-types
 - git diff --exit-code
 script:
-- xvfb-run wct -l chrome
-- xvfb-run wct -l firefox
-- if [ "${TRAVIS_PULL_REQUEST}" = "false" ]; then wct -s 'windows 10/microsoftedge@14' -s 'windows 10/microsoftedge@15' -s 'windows 8.1/internet explorer@11' -s 'os x 10.11/safari@9' -s 'macos 10.12/safari@10' -s 'macos 10.12/safari@11' -s 'Linux/chrome@41'; fi
+- wct -l chrome -l firefox
+- if [ "${TRAVIS_PULL_REQUEST}" = "false" ]; then travis_wait 30 ./util/travis-sauce-test.sh; fi
 env:
   global:
   - secure: bfF/o1ewpOxDNqTzWfvlwgRgGfP8OXhSQLLdEwZ6izO9tckMJuSNghk3qBXCEQJwTcUEyXP6EqfzIrRAvDXPa0H3OoinbrooDyV2wIDaVRK++WR2iZIqzqo3hGOdzm4tdrGJZe5av5Rk661Hls8aPfLbjdzcGuYXi8B4wZq2xMI=

lib/elements/dom-repeat.html

@@ -473,7 +473,6 @@
           this.__debounceRender(this.__render, this.delay);
         } else if (this.__observePaths) {
           // Otherwise, re-render if the path changed matches an observed path
-          path = path.substring(path.indexOf('.') + 1);
           let paths = this.__observePaths;
           for (let i=0; i<paths.length; i++) {
             if (path.indexOf(paths[i]) === 0) {
@@ -558,7 +557,7 @@
         let itemIdx = isntIdxToItemsIdx[instIdx];
         let item = items[itemIdx];
         itemsIdxToInstIdx[itemIdx] = instIdx;
-        if (inst && instIdx < this.__limit) {
+        if (inst) {
           inst._setPendingProperty(this.as, item);
           inst._setPendingProperty(this.indexAs, instIdx);
           inst._setPendingProperty(this.itemsIndexAs, itemIdx);

lib/legacy/class.html

@@ -34,7 +34,7 @@
      * the underlying element.
      *
      * @template T
-     * @param {!(PolymerInit|Array<!PolymerInit>)} behaviors Behavior object or array of behaviors.
+     * @param {!Object|!Array<!Object>} behaviors Behavior object or array of behaviors.
      * @param {function(new:T)} klass Element class.
      * @return {function(new:T)} Returns a new Element class extended by the
      * passed in `behaviors` and also by `Polymer.LegacyElementMixin`.

lib/legacy/legacy-element-mixin.html

@@ -937,7 +937,7 @@
        */
       _logger(level, args) {
         // accept ['foo', 'bar'] and [['foo', 'bar']]
-        if (Array.isArray(args) && args.length === 1) {
+        if (Array.isArray(args) && args.length === 1 && Array.isArray(args[0])) {
           args = args[0];
         }
         switch(level) {

lib/legacy/polymer.dom.html

@@ -345,7 +345,7 @@
   /**
    * @function
    * @param {string} selector
-   * @return {!NodeList}
+   * @return {!NodeList<!Element>}
    */
   Polymer.DomApi.prototype.querySelectorAll;
 

lib/utils/templatize.html

@@ -408,7 +408,7 @@
       /**
        * Returns an anonymous `Polymer.PropertyEffects` class bound to the
        * `<template>` provided.  Instancing the class will result in the
-       * template being stamped into document fragment stored as the instance's
+       * template being stamped into a document fragment stored as the instance's
        * `root` property, after which it can be appended to the DOM.
        *
        * Templates may utilize all Polymer data-binding features as well as