Remove @Suppress {missingProperties}

It may break stuff in closure to use this much, so try to shape classes
for closure directly.

Use polymer pass v2 to remove warnings for properties defined in
`properties` block

Author: dfreedm

closure.log

@@ -61,15 +61,18 @@ Polymer.sanitizeDOMValue;
  */
 function JSCompiler_renameProperty(string, obj) {}
 
-/** @record */
-function PolymerTelemetry(){}
-/** @type {number} */
-PolymerTelemetry.prototype.instanceCount;
-/** @type {Array<HTMLElement>} */
-PolymerTelemetry.prototype.registrations;
-/** @type {function(HTMLElement)} */
-PolymerTelemetry.prototype._regLog;
-/** @type {function(HTMLElement)} */
-PolymerTelemetry.prototype.register;
-/** @type {function(HTMLElement)} */
-PolymerTelemetry.prototype.dumpRegistrations;
+/** @type {Object} */
+Polymer.telemetry;
+
+/** @type {Object} */
+Polymer_PropertyEffects.prototype.__computeEffects;
+/** @type {Object} */
+Polymer_PropertyEffects.prototype.__reflectEffects;
+/** @type {Object} */
+Polymer_PropertyEffects.prototype.__notifyEffects;
+/** @type {Object} */
+Polymer_PropertyEffects.prototype.__propagateEffects;
+/** @type {Object} */
+Polymer_PropertyEffects.prototype.__observeEffects;
+/** @type {Object} */
+Polymer_PropertyEffects.prototype.__readOnly;

externs/polymer-externs.js

@@ -2,6 +2,7 @@
  * @fileoverview Externs for webcomponents polyfills
  * @externs
  */
+/* eslint-disable */
 
 let HTMLImports = {
   /**

externs/webcomponents-externs.js

@@ -107,6 +107,7 @@ gulp.task('closure', ['clean'], () => {
   }
 
   config('polymer.html');
+  // config('lib/mixins/property-effects.html');
 
   const project = new PolymerProject({
     shell: `./${entry}`,
@@ -146,16 +147,19 @@ gulp.task('closure', ['clean'], () => {
     rewrite_polyfills: false,
     new_type_inf: true,
     checks_only: CLOSURE_LINT_ONLY,
+    polymer_version: 2,
     externs: [
       'bower_components/shadycss/externs/shadycss-externs.js',
       'externs/webcomponents-externs.js',
       'externs/closure-types.js',
       'externs/polymer-externs.js',
     ],
     extra_annotation_name: [
-      'polymerMixin',
-      'polymerMixinClass',
-      'polymerElement'
+      'appliesMixin',
+      'mixinClass',
+      'mixinFunction',
+      'polymer',
+      'customElement'
     ]
   }, closurePluginOptions);
 

gulpfile.js

@@ -32,7 +32,6 @@
    *
    * @mixinFunction
    * @polymer
-   * @appliesMixin Polymer.ElementMixin
    * @memberof Polymer
    * @summary Element mixin for recording dynamic associations between item paths in a
    * master `items` array and a `selected` array
@@ -48,6 +47,7 @@
 
     /**
      * @mixinClass
+     * @polymer
      * @implements {Polymer_ArraySelectorMixin}
      * @unrestricted
      */

lib/elements/array-selector.html

@@ -87,7 +87,6 @@
       /**
        * Forces the element to render its content. This is typically only
        * necessary to call if HTMLImports with the async attribute are used.
-       * @suppress {missingProperties}
        */
       render() {
         let template;

lib/elements/dom-bind.html

@@ -230,7 +230,6 @@
       return true;
     }
 
-    /** @suppress {missingProperties} */
     __syncHostProperties() {
       let props = this.__invalidProps;
       if (props) {

lib/elements/dom-if.html

@@ -118,7 +118,7 @@
    * @polymer
    * @memberof Polymer
    * @extends {domRepeatBase}
-   * @appliesMixin Polymer.MutableData
+   * @appliesMixin Polymer.OptionalMutableData
    * @summary Custom element for stamping instance of a template bound to
    *   items in an array.
    */
@@ -372,10 +372,6 @@
       return true;
     }
 
-    /* eslint-disable valid-jsdoc */
-    /**
-     * @suppress {missingProperties}
-     */
     __getMethodHost() {
       // Technically this should be the owner of the outermost template.
       // In shadow dom, this is always getRootNode().host, but we can
@@ -384,7 +380,6 @@
       // instance causing it to get a dataHost.
       return this.__dataHost._methodHost || this.__dataHost;
     }
-    /* eslint-enable valid-jsdoc */
 
     __sortChanged(sort) {
       let methodHost = this.__getMethodHost();

lib/elements/dom-repeat.html

@@ -135,7 +135,7 @@
      * @param {!PolymerInit} info Polymer info object
      * @param {!HTMLElement} Base base class to extend with info object
      * @return {!HTMLElement} Generated class
-     * @suppress {missingProperties, checkTypes}
+     * @suppress {checkTypes}
      * @private
      */
     function GenerateClassFromInfo(info, Base) {
@@ -151,7 +151,6 @@
         }
 
         /**
-         * @suppress {missingProperties}
          * @return {HTMLTemplateElement} template for this class
          */
         static get template() {

lib/legacy/class.html

@@ -459,7 +459,6 @@
        * a user should call `distributeContent` if distribution has been
        * invalidated due to an element being added or removed from the shadowRoot
        * that contains an insertion point (`<slot>`) inside its subtree.
-       * @suppress {missingProperties}
        */
       distributeContent() {
         if (window.ShadyDOM && this.shadowRoot) {