Enable ESLint 'semi' rule

Closes #4651

.eslintrc.json

@@ -6,6 +6,7 @@
   "rules": {
     "no-console": "off",
     "no-var": "error",
+    "semi": "error",
     "strict": "error",
     "valid-jsdoc": ["error", {
       "requireReturn": false,

lib/elements/array-selector.html

@@ -105,11 +105,11 @@
             value: false
           }
 
-        }
+        };
       }
 
       static get observers() {
-        return ['__updateSelection(multi, items.*)']
+        return ['__updateSelection(multi, items.*)'];
       }
 
       constructor() {
@@ -222,7 +222,7 @@
         // order: selected array index
         this.__selectedMap = new Map();
         // Initialize selection
-        this.selected = this.multi ? [] : null
+        this.selected = this.multi ? [] : null;
         this.selectedItem = null;
       }
 
@@ -416,7 +416,7 @@
   class ArraySelector extends baseArraySelector {
     // Not needed to find template; can be removed once the analyzer
     // can find the tag name from customElements.define call
-    static get is() { return 'array-selector' }
+    static get is() { return 'array-selector'; }
   }
   customElements.define(ArraySelector.is, ArraySelector);
   Polymer.ArraySelector = ArraySelector;

lib/elements/dom-bind.html

@@ -52,7 +52,7 @@
      */
     class DomBind extends domBindBase {
 
-      static get observedAttributes() { return ['mutable-data'] }
+      static get observedAttributes() { return ['mutable-data']; }
 
       constructor() {
         super();
@@ -104,7 +104,7 @@
               } else {
                 throw new Error('dom-bind requires a <template> child');
               }
-            })
+            });
             observer.observe(this, {childList: true});
             return;
           }

lib/elements/dom-if.html

@@ -80,7 +80,7 @@
           observer: '__debounceRender'
         }
 
-      }
+      };
 
     }
 
@@ -181,7 +181,7 @@
               } else {
                 throw new Error('dom-if requires a <template> child');
               }
-            })
+            });
             observer.observe(this, {childList: true});
             return false;
           }

lib/elements/dom-module.html

@@ -53,7 +53,7 @@
    */
   class DomModule extends HTMLElement {
 
-    static get observedAttributes() { return ['id'] }
+    static get observedAttributes() { return ['id']; }
 
     /**
      * Retrieves the element specified by the css `selector` in the module

lib/elements/dom-repeat.html

@@ -267,12 +267,12 @@
           computed: '__computeFrameTime(targetFramerate)'
         }
 
-      }
+      };
 
     }
 
     static get observers() {
-      return [ '__itemsChanged(items.*)' ]
+      return [ '__itemsChanged(items.*)' ];
     }
 
     constructor() {
@@ -327,7 +327,7 @@
             } else {
               throw new Error('dom-repeat requires a <template> child');
             }
-          })
+          });
           observer.observe(this, {childList: true});
           return false;
         }

lib/legacy/class.html

@@ -24,7 +24,7 @@
       attributeChanged: true,
       // meta objects
       behaviors: true
-    }
+    };
 
     /**
      * Applies a "legacy" behavior or array of behaviors to the provided class.
@@ -240,7 +240,7 @@
        }
       }
 
-      PolymerGenerated.generatedFrom = info
+      PolymerGenerated.generatedFrom = info;
 
       for (let p in info) {
         // NOTE: cannot copy `metaProps` methods onto prototype at least because
@@ -335,7 +335,7 @@
       // decorate klass with registration info
       klass.is = info.is;
       return klass;
-    }
+    };
 
     Polymer.mixinBehaviors = mixinBehaviors;
 

lib/legacy/legacy-element-mixin.html

@@ -369,7 +369,7 @@
         });
         event.detail = detail;
         let node = options.node || this;
-        node.dispatchEvent(event)
+        node.dispatchEvent(event);
         return event;
       }
 
@@ -687,7 +687,7 @@
        * @param {string} jobName The name of the debouncer started with `debounce`
        */
       cancelDebouncer(jobName) {
-        this._debouncers = this._debouncers || {}
+        this._debouncers = this._debouncers || {};
         let debouncer = this._debouncers[jobName];
         if (debouncer) {
           debouncer.cancel();
@@ -927,7 +927,7 @@
        * @param {...*} args Array of strings or objects to log
        */
       _error(...args) {
-        this._logger('error', args)
+        this._logger('error', args);
       }
 
       /**

lib/legacy/polymer.dom.html

@@ -34,7 +34,7 @@
    */
   const matchesSelector = function(node, selector) {
     return normalizedMatchesSelector.call(node, selector);
-  }
+  };
 
   /**
    * Node API wrapper class returned from `Polymer.dom.(target)` when
@@ -196,7 +196,7 @@
       let method = methods[i];
       proto[method] = /** @this {DomApi} */ function() {
         return this.node[method].apply(this.node, arguments);
-      }
+      };
     }
   }
 

lib/mixins/element-mixin.html

@@ -808,7 +808,7 @@
      * @private
      */
     _regLog: function(prototype) {
-      console.log('[' + prototype.is + ']: registered')
+      console.log('[' + prototype.is + ']: registered');
     },
     /**
      * Registers a class prototype for telemetry purposes.

lib/mixins/property-accessors.html

@@ -468,7 +468,7 @@
        */
       _setPendingProperty(property, value) {
         let old = this.__data[property];
-        let changed = this._shouldPropertyChange(property, value, old)
+        let changed = this._shouldPropertyChange(property, value, old);
         if (changed) {
           if (!this.__dataPending) {
             this.__dataPending = {};
@@ -528,7 +528,7 @@
             this._initializeInstanceProperties(this.__dataInstanceProps);
             this.__dataInstanceProps = null;
           }
-          this.ready()
+          this.ready();
         }
       }
 

lib/mixins/property-effects.html

@@ -42,7 +42,7 @@
     PROPAGATE: '__propagateEffects',
     OBSERVE: '__observeEffects',
     READ_ONLY: '__readOnly'
-  }
+  };
 
   /**
    * @typedef {{
@@ -1979,7 +1979,7 @@
         if (protectedSetter) {
           this['_set' + upper(property)] = /** @this {PropertyEffects} */function(value) {
             this._setProperty(property, value);
-          }
+          };
         }
       }
 
@@ -2687,7 +2687,7 @@
       }
     }
 
-  }
+  };
 
 })();
 </script>

lib/utils/array-splice.html

@@ -309,7 +309,7 @@
      * of items added at this location.
      */
     calculateSplices
-  }
+  };
 
 })();
 </script>

lib/utils/async.html

@@ -33,7 +33,7 @@
         try {
           cb();
         } catch (e) {
-          setTimeout(() => { throw e });
+          setTimeout(() => { throw e; });
         }
       }
     }
@@ -72,9 +72,9 @@
        */
       after(delay) {
         return  {
-          run(fn) { return setTimeout(fn, delay) },
+          run(fn) { return setTimeout(fn, delay); },
           cancel: window.clearTimeout.bind(window)
-        }
+        };
       },
       /**
        * Enqueues a function called in the next task.

lib/utils/boot.html

@@ -27,7 +27,7 @@
    */
   window.Polymer = function(info) {
     return window.Polymer._polymerFn(info);
-  }
+  };
 
   // support user settings on the Polymer object
   if (userPolymer) {
@@ -46,7 +46,7 @@
    */
   window.Polymer._polymerFn = function(info) { // eslint-disable-line no-unused-vars
     throw new Error('Load polymer.html to use the Polymer() function.');
-  }
+  };
   /* eslint-enable */
 
   window.Polymer.version = '2.0.1';
@@ -58,7 +58,7 @@
   */
   window.JSCompiler_renameProperty = function(prop, obj) {
     return prop;
-  }
+  };
   /* eslint-enable */
 
 })();

lib/utils/debounce.html

@@ -41,7 +41,7 @@
       this._callback = callback;
       this._timer = this._asyncModule.run(() => {
         this._timer = null;
-        this._callback()
+        this._callback();
       });
     }
     /**

lib/utils/flattened-nodes-observer.html

@@ -95,7 +95,7 @@
       /** @type {function()} */
       this._boundSchedule = () => {
         this._schedule();
-      }
+      };
       this.connect();
       this._schedule();
     }

lib/utils/flush.html

@@ -22,7 +22,7 @@
    */
   Polymer.enqueueDebouncer = function(debouncer) {
     debouncerQueue.push(debouncer);
-  }
+  };
 
   function flushDebouncers() {
     const didFlush = Boolean(debouncerQueue.length);
@@ -54,7 +54,7 @@
       }
       debouncers = flushDebouncers();
     } while (shadyDOM || debouncers);
-  }
+  };
 
 })();
 </script>

lib/utils/gestures.html

@@ -45,7 +45,7 @@
   let SUPPORTS_PASSIVE = false;
   (function() {
     try {
-      let opts = Object.defineProperty({}, 'passive', {get: function() {SUPPORTS_PASSIVE = true;}})
+      let opts = Object.defineProperty({}, 'passive', {get: function() {SUPPORTS_PASSIVE = true;}});
       window.addEventListener('test', null, opts);
       window.removeEventListener('test', null, opts);
     } catch(e) {}

lib/utils/import-href.html

@@ -64,7 +64,7 @@
     let cleanup = function() {
       link.removeEventListener('load', loadListener);
       link.removeEventListener('error', errorListener);
-    }
+    };
     let loadListener = function(event) {
       cleanup();
       // In case of a successful load, cache the load event on the link so

lib/utils/render-status.html

@@ -52,7 +52,7 @@
     } catch(e) {
       setTimeout(() => {
         throw e;
-      })
+      });
     }
   }
 

lib/utils/settings.html

@@ -60,7 +60,7 @@
    */
   Polymer.setRootPath = function(path) {
     Polymer.rootPath = path;
-  }
+  };
 
   /**
    * A global callback used to sanitize any value before inserting it into the DOM. The callback signature is:
@@ -92,6 +92,6 @@
    */
   Polymer.setSanitizeDOMValue = function(newSanitizeDOMValue) {
     Polymer.sanitizeDOMValue = newSanitizeDOMValue;
-  }
+  };
 })();
 </script>