From 6d636138605ec8c629581388ab06722374f94a25 Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Fri, 14 Apr 2017 15:00:27 -0700
Subject: [PATCH] Remove unused @method

---
 lib/elements/array-selector.html     |  7 ------
 lib/elements/dom-module.html         |  2 --
 lib/elements/dom-repeat.html         |  3 ---
 lib/legacy/legacy-element-mixin.html | 32 ----------------------------
 lib/legacy/templatizer-behavior.html |  1 -
 lib/mixins/element-mixin.html        |  1 -
 lib/mixins/property-accessors.html   |  6 ------
 lib/mixins/property-effects.html     |  9 --------
 lib/utils/gestures.html              |  1 -
 lib/utils/import-href.html           |  1 -
 lib/utils/templatize.html            |  1 -
 11 files changed, 64 deletions(-)

diff --git a/lib/elements/array-selector.html b/lib/elements/array-selector.html
index 5cf7c3c8be..bff749c6aa 100644
--- a/lib/elements/array-selector.html
+++ b/lib/elements/array-selector.html
@@ -193,7 +193,6 @@
       /**
        * Clears the selection state.
        *
-       * @method clearSelection
        */
       clearSelection() {
         // Unbind previous selection
@@ -211,7 +210,6 @@
       /**
        * Returns whether the item is currently selected.
        *
-       * @method isSelected
        * @param {*} item Item from `items` array to test
        * @return {boolean} Whether the item is selected
        */
@@ -222,7 +220,6 @@
       /**
        * Returns whether the item is currently selected.
        *
-       * @method isSelected
        * @param {*} idx Index from `items` array to test
        * @return {boolean} Whether the item is selected
        */
@@ -252,7 +249,6 @@
       /**
        * Deselects the given item if it is already selected.
        *
-       * @method deselect
        * @param {*} item Item from `items` array to deselect
        */
       deselect(item) {
@@ -275,7 +271,6 @@
       /**
        * Deselects the given index if it is already selected.
        *
-       * @method deselect
        * @param {number} idx Index from `items` array to deselect
        */
       deselectIndex(idx) {
@@ -286,7 +281,6 @@
        * Selects the given item.  When `toggle` is true, this will automatically
        * deselect the item if already selected.
        *
-       * @method select
        * @param {*} item Item from `items` array to select
        */
       select(item) {
@@ -297,7 +291,6 @@
        * Selects the given index.  When `toggle` is true, this will automatically
        * deselect the item if already selected.
        *
-       * @method select
        * @param {number} idx Index from `items` array to select
        */
       selectIndex(idx) {
diff --git a/lib/elements/dom-module.html b/lib/elements/dom-module.html
index 29c0c1ae09..99edb7f109 100644
--- a/lib/elements/dom-module.html
+++ b/lib/elements/dom-module.html
@@ -56,7 +56,6 @@
     /**
      * Retrieves the element specified by the css `selector` in the module
      * registered by `id`. For example, this.import('foo', 'img');
-     * @method import
      * @param {string} id The id of the dom-module in which to search.
      * @param {string=} selector The css selector by which to find the element.
      * @return {Element} Returns the element which matches `selector` in the
@@ -108,7 +107,6 @@
      * Registers the dom-module at a given id. This method should only be called
      * when a dom-module is imperatively created. For
      * example, `document.createElement('dom-module').register('foo')`.
-     * @method register
      * @param {string=} id The id at which to register the dom-module.
      */
     register(id) {
diff --git a/lib/elements/dom-repeat.html b/lib/elements/dom-repeat.html
index 4a8ac93d84..c791e66634 100644
--- a/lib/elements/dom-repeat.html
+++ b/lib/elements/dom-repeat.html
@@ -626,7 +626,6 @@
      * `modelForElement(el).set('item.<sub-prop>', value)`
      * should be used.
      *
-     * @method itemForElement
      * @param {HTMLElement} el Element for which to return the item.
      * @return {*} Item associated with the element.
      */
@@ -640,7 +639,6 @@
      * If `sort` is provided, the index will reflect the sorted order (rather
      * than the original array order).
      *
-     * @method indexForElement
      * @param {HTMLElement} el Element for which to return the index.
      * @return {*} Row index associated with the element (note this may
      *   not correspond to the array index if a user `sort` is applied).
@@ -663,7 +661,6 @@
      *     model.set('item.checked', true);
      *   }
      *
-     * @method modelForElement
      * @param {HTMLElement} el Element for which to return a template model.
      * @return {TemplateInstanceBase} Model representing the binding scope for
      *   the element.
diff --git a/lib/legacy/legacy-element-mixin.html b/lib/legacy/legacy-element-mixin.html
index 2add4236f4..a47de0b5fa 100644
--- a/lib/legacy/legacy-element-mixin.html
+++ b/lib/legacy/legacy-element-mixin.html
@@ -230,7 +230,6 @@
        * Copies own properties (including accessor descriptors) from a source
        * object to a target object.
        *
-       * @method extend
        * @param {Object} prototype Target object to copy properties to.
        * @param {Object} api Source object to copy properties from.
        * @return {Object} prototype object that was passed as first argument.
@@ -256,7 +255,6 @@
        * properties.  To ensure only `ownProperties` are copied from source
        * to target and that accessor implementations are copied, use `extend`.
        *
-       * @method mixin
        * @param {Object} target Target object to copy properties to.
        * @param {Object} source Source object to copy properties from.
        * @return {Object} Target object that was passed as first argument.
@@ -292,7 +290,6 @@
        * Calls `importNode` on the `content` of the `template` specified and
        * returns a document fragment containing the imported content.
        *
-       * @method instanceTemplate
        * @param {HTMLTemplateElement} template HTML template element to instance.
        * @return {DocumentFragment} Document fragment containing the imported
        *   template content.
@@ -309,7 +306,6 @@
       /**
        * Dispatches a custom event with an optional detail value.
        *
-       * @method fire
        * @param {string} type Name of event type.
        * @param {*=} detail Detail value containing event-specific
        *   payload.
@@ -337,7 +333,6 @@
        * Convenience method to add an event listener on a given element,
        * late bound to a named method on this element.
        *
-       * @method listen
        * @param {Element} node Element to add event listener to.
        * @param {string} eventName Name of event to listen for.
        * @param {string} methodName Name of handler method on `this` to call.
@@ -362,7 +357,6 @@
        * Convenience method to remove an event listener from a given element,
        * late bound to a named method on this element.
        *
-       * @method unlisten
        * @param {Element} node Element to remove event listener from.
        * @param {string} eventName Name of event to stop listening to.
        * @param {string} methodName Name of handler method on `this` to not call
@@ -388,7 +382,6 @@
        *   - 'y': scroll only in the 'y' direction
        *   - 'none': disable scrolling for this node
        *
-       * @method setScrollDirection
        * @param {string=} direction Direction to allow scrolling
        * Defaults to `all`.
        * @param {HTMLElement=} node Element to apply scroll direction setting.
@@ -404,7 +397,6 @@
        *
        * This function calls `Polymer.dom(this.root).querySelector(slctr)`.
        *
-       * @method $$
        * @param {string} slctr Selector to run on this local DOM scope
        * @return {Element} Element found by the selector, or null if not found.
        */
@@ -429,7 +421,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.
-       * @method distributeContent
        */
       distributeContent() {
         if (window.ShadyDOM && this.shadowRoot) {
@@ -443,7 +434,6 @@
        * any `<content>` elements are replaced with the list of nodes distributed
        * to the `<content>`, the result of its `getDistributedNodes` method.
        *
-       * @method getEffectiveChildNodes
        * @return {Array<Node>} List of effctive child nodes.
        */
       getEffectiveChildNodes() {
@@ -454,7 +444,6 @@
        * Returns a list of nodes distributed within this element that match
        * `selector`. These can be dom children or elements distributed to
        * children that are insertion points.
-       * @method queryDistributedElements
        * @param {string} selector Selector to run.
        * @return {Array<Node>} List of distributed elements that match selector.
        */
@@ -468,7 +457,6 @@
        * any `<content>` elements are replaced with the list of elements
        * distributed to the `<content>`.
        *
-       * @method getEffectiveChildren
        * @return {Array<Node>} List of effctive children.
        */
       getEffectiveChildren() {
@@ -483,7 +471,6 @@
        * text content's of the element's effective childNodes (the elements
        * returned by <a href="#getEffectiveChildNodes>getEffectiveChildNodes</a>.
        *
-       * @method getEffectiveTextContent
        * @return {string} List of effctive children.
        */
       getEffectiveTextContent() {
@@ -501,7 +488,6 @@
        * Returns the first effective childNode within this element that
        * match `selector`. These can be dom child nodes or elements distributed
        * to children that are insertion points.
-       * @method queryEffectiveChildren
        * @param {string} selector Selector to run.
        * @return {Object<Node>} First effective child node that matches selector.
        */
@@ -514,7 +500,6 @@
        * Returns a list of effective childNodes within this element that
        * match `selector`. These can be dom child nodes or elements distributed
        * to children that are insertion points.
-       * @method queryEffectiveChildren
        * @param {string} selector Selector to run.
        * @return {Array<Node>} List of effective child nodes that match selector.
        */
@@ -528,7 +513,6 @@
        * If this element contains more than one `<slot>` in its local DOM,
        * an optional selector may be passed to choose the desired content.
        *
-       * @method getContentChildNodes
        * @param {string=} slctr CSS selector to choose the desired
        *   `<slot>`.  Defaults to `content`.
        * @return {Array<Node>} List of distributed nodes for the `<slot>`.
@@ -547,7 +531,6 @@
        * content.  This method differs from `getContentChildNodes` in that only
        * elements are returned.
        *
-       * @method getContentChildNodes
        * @param {string=} slctr CSS selector to choose the desired
        *   `<content>`.  Defaults to `content`.
        * @return {Array<HTMLElement>} List of distributed nodes for the
@@ -562,7 +545,6 @@
       /**
        * Checks whether an element is in this element's light DOM tree.
        *
-       * @method isLightDescendant
        * @param {?Node} node The element to be checked.
        * @return {boolean} true if node is in this element's light DOM tree.
        */
@@ -574,7 +556,6 @@
       /**
        * Checks whether an element is in this element's local DOM tree.
        *
-       * @method isLocalDescendant
        * @param {HTMLElement=} node The element to be checked.
        * @return {boolean} true if node is in this element's local DOM tree.
        */
@@ -611,7 +592,6 @@
        *       } 100);
        *     }
        *
-       * @method debounce
        * @param {string} jobName String to indentify the debounce job.
        * @param {function()} callback Function that is called (with `this`
        *   context) when the wait time elapses.
@@ -634,7 +614,6 @@
       /**
        * Returns whether a named debouncer is active.
        *
-       * @method isDebouncerActive
        * @param {string} jobName The name of the debouncer started with `debounce`
        * @return {boolean} Whether the debouncer is active (has not yet fired).
        */
@@ -647,7 +626,6 @@
       /**
        * Immediately calls the debouncer `callback` and inactivates it.
        *
-       * @method flushDebouncer
        * @param {string} jobName The name of the debouncer started with `debounce`
        */
       flushDebouncer(jobName) {
@@ -661,7 +639,6 @@
       /**
        * Cancels an active debouncer.  The `callback` will not be called.
        *
-       * @method cancelDebouncer
        * @param {string} jobName The name of the debouncer started with `debounce`
        */
       cancelDebouncer(jobName) {
@@ -678,7 +655,6 @@
        * By default (if no waitTime is specified), async callbacks are run at
        * microtask timing, which will occur before paint.
        *
-       * @method async
        * @param {Function} callback The callback function to run, bound to `this`.
        * @param {number=} waitTime Time to wait before calling the
        *   `callback`.  If unspecified or 0, the callback will be run at microtask
@@ -693,7 +669,6 @@
       /**
        * Cancels an async operation started with `async`.
        *
-       * @method cancelAsync
        * @param {number} handle Handle returned from original `async` call to
        *   cancel.
        */
@@ -707,7 +682,6 @@
       /**
        * Convenience method for creating an element and configuring it.
        *
-       * @method create
        * @param {string} tag HTML element tag to create.
        * @param {Object} props Object of properties to configure on the
        *    instance.
@@ -735,7 +709,6 @@
        * In the `onload` callback, the `import` property of the `link`
        * element will contain the imported document contents.
        *
-       * @method importHref
        * @param {string} href URL to document to load.
        * @param {Function} onload Callback to notify when an import successfully
        *   loaded.
@@ -755,7 +728,6 @@
        * Polyfill for Element.prototype.matches, which is sometimes still
        * prefixed.
        *
-       * @method elementMatches
        * @param {string} selector Selector to test.
        * @param {Element=} node Element to test the selector against.
        * @return {boolean} Whether the element matches the selector.
@@ -767,7 +739,6 @@
       /**
        * Toggles an HTML attribute on or off.
        *
-       * @method toggleAttribute
        * @param {string} name HTML attribute name
        * @param {boolean=} bool Boolean to force the attribute on or off.
        *    When unspecified, the state of the attribute will be reversed.
@@ -789,7 +760,6 @@
       /**
        * Toggles a CSS class on or off.
        *
-       * @method toggleClass
        * @param {string} name CSS class name
        * @param {boolean=} bool Boolean to force the class on or off.
        *    When unspecified, the state of the class will be reversed.
@@ -824,7 +794,6 @@
        * Cross-platform helper for setting an element's CSS `translate3d`
        * property.
        *
-       * @method translate3d
        * @param {number} x X offset.
        * @param {number} y Y offset.
        * @param {number} z Z offset.
@@ -846,7 +815,6 @@
        * If the array is passed directly, **no change
        * notification is generated**.
        *
-       * @method arrayDelete
        * @param {string | !Array<number|string>} arrayOrPath Path to array from which to remove the item
        *   (or the array itself).
        * @param {*} item Item to remove.
diff --git a/lib/legacy/templatizer-behavior.html b/lib/legacy/templatizer-behavior.html
index 7a15533b65..4f3a14aa85 100644
--- a/lib/legacy/templatizer-behavior.html
+++ b/lib/legacy/templatizer-behavior.html
@@ -121,7 +121,6 @@
        * instance the element is contained in.  A template model should be used
        * to manipulate data associated with this template instance.
        *
-       * @method modelForElement
        * @param {HTMLElement} el Element for which to return a template model.
        * @return {TemplateInstanceBase} Model representing the binding scope for
        *   the element.
diff --git a/lib/mixins/element-mixin.html b/lib/mixins/element-mixin.html
index 6f5e2d8739..a8b6a0d8c2 100644
--- a/lib/mixins/element-mixin.html
+++ b/lib/mixins/element-mixin.html
@@ -677,7 +677,6 @@
        * However, this method may be overridden to allow an element
        * to put its dom in another location.
        *
-       * @method _attachDom
        * @throws {Error}
        * @suppress {missingReturn}
        * @param {NodeList} dom to attach to the element.
diff --git a/lib/mixins/property-accessors.html b/lib/mixins/property-accessors.html
index 609b31c2cc..ca9b9dea24 100644
--- a/lib/mixins/property-accessors.html
+++ b/lib/mixins/property-accessors.html
@@ -193,7 +193,6 @@
        * assigns the given value to the attribute.
        *
        *
-       * @method _ensureAttribute
        * @param {string} attribute Name of attribute to ensure is set.
        * @param {string} value of the attribute.
        */
@@ -209,7 +208,6 @@
        * This method calls the `_deserializeValue` method to convert the string to
        * a typed value.
        *
-       * @method _attributeToProperty
        * @param {string} attribute Name of attribute to deserialize.
        * @param {string} value of the attribute.
        * @param {*} type type to deserialize to.
@@ -225,7 +223,6 @@
       /**
        * Serializes a property to its associated attribute.
        *
-       * @method _propertyToAttribute
        * @param {string} property Property name to reflect.
        * @param {string=} attribute Attribute name to reflect.
        * @param {*=} value Property value to refect.
@@ -246,7 +243,6 @@
        * the attribute will be removed (this is the default for boolean
        * type `false`).
        *
-       * @method _valueToNodeAttribute
        * @param {Element} node Element to set attribute to.
        * @param {*} value Value to serialize.
        * @param {string} attribute Attribute name to serialize to.
@@ -267,7 +263,6 @@
        * HTML attributes.  Users may override this method on Polymer element
        * prototypes to provide serialization for custom types.
        *
-       * @method _serializeValue
        * @param {*} value Property value to serialize.
        * @return {string | undefined} String serialized from the provided property value.
        */
@@ -306,7 +301,6 @@
        * Note: The return value of `undefined` is used as a sentinel value to
        * indicate the attribute should be removed.
        *
-       * @method _deserializeValue
        * @param {string} value Attribute value to deserialize.
        * @param {*} type Type to deserialize the string to.
        * @return {*} Typed value deserialized from the provided string.
diff --git a/lib/mixins/property-effects.html b/lib/mixins/property-effects.html
index 92537dcac5..2db0fbffa6 100644
--- a/lib/mixins/property-effects.html
+++ b/lib/mixins/property-effects.html
@@ -1588,7 +1588,6 @@
        * Aliases one data path as another, such that path notifications from one
        * are routed to the other.
        *
-       * @method linkPaths
        * @param {string | !Array<string|number>} to Target path to link.
        * @param {string | !Array<string|number>} from Source path to link.
        * @public
@@ -1606,7 +1605,6 @@
        * Note, the path to unlink should be the target (`to`) used when
        * linking the paths.
        *
-       * @method unlinkPaths
        * @param {string | !Array<string|number>} path Target path to unlink.
        * @public
        */
@@ -1659,7 +1657,6 @@
        * `undefined` (this method does not throw when dereferencing undefined
        * paths).
        *
-       * @method get
        * @param {(string|!Array<(string|number)>)} path Path to the value
        *   to read.  The path may be specified as a string (e.g. `foo.bar.baz`)
        *   or an array of path parts (e.g. `['foo.bar', 'baz']`).  Note that
@@ -1684,7 +1681,6 @@
        * this method does nothing (this method does not throw when
        * dereferencing undefined paths).
        *
-       * @method set
        * @param {(string|!Array<(string|number)>)} path Path to the value
        *   to write.  The path may be specified as a string (e.g. `'foo.bar.baz'`)
        *   or an array of path parts (e.g. `['foo.bar', 'baz']`).  Note that
@@ -1718,7 +1714,6 @@
        * This method notifies other paths to the same array that a
        * splice occurred to the array.
        *
-       * @method push
        * @param {string} path Path to array.
        * @param {...*} items Items to push onto array
        * @return {number} New length of the array.
@@ -1744,7 +1739,6 @@
        * This method notifies other paths to the same array that a
        * splice occurred to the array.
        *
-       * @method pop
        * @param {string} path Path to array.
        * @return {*} Item that was removed.
        * @public
@@ -1770,7 +1764,6 @@
        * This method notifies other paths to the same array that a
        * splice occurred to the array.
        *
-       * @method splice
        * @param {string} path Path to array.
        * @param {number} start Index from which to start removing/inserting.
        * @param {number} deleteCount Number of items to remove.
@@ -1806,7 +1799,6 @@
        * This method notifies other paths to the same array that a
        * splice occurred to the array.
        *
-       * @method shift
        * @param {string} path Path to array.
        * @return {*} Item that was removed.
        * @public
@@ -1831,7 +1823,6 @@
        * This method notifies other paths to the same array that a
        * splice occurred to the array.
        *
-       * @method unshift
        * @param {string} path Path to array.
        * @param {...*} items Items to insert info array
        * @return {number} New length of the array.
diff --git a/lib/utils/gestures.html b/lib/utils/gestures.html
index 4fe82a9ac0..4ee65c3b87 100644
--- a/lib/utils/gestures.html
+++ b/lib/utils/gestures.html
@@ -562,7 +562,6 @@
      * This method should only be called during testing with simulated touch inputs.
      * Calling this method in production may cause duplicate taps or other Gestures.
      *
-     * @method resetMouseCanceller
      * @memberof Polymer.Gestures
      */
     resetMouseCanceller: function() {
diff --git a/lib/utils/import-href.html b/lib/utils/import-href.html
index 627eae9c9b..77933a4e43 100644
--- a/lib/utils/import-href.html
+++ b/lib/utils/import-href.html
@@ -35,7 +35,6 @@
    * element will contain the imported document contents.
    *
    * @memberof Polymer
-   * @method importHref
    * @param {string} href URL to document to load.
    * @param {Function=} onload Callback to notify when an import successfully
    *   loaded.
diff --git a/lib/utils/templatize.html b/lib/utils/templatize.html
index 5152176ee3..4a6ad4312a 100644
--- a/lib/utils/templatize.html
+++ b/lib/utils/templatize.html
@@ -431,7 +431,6 @@
        *   }
        *
        * @memberof Polymer.Templatize
-       * @method modelForElement
        * @param {HTMLTemplateElement} template The model will be returned for
        *   elements stamped from this template
        * @param {HTMLElement} el Element for which to return a template model.