From cee1893b9e441b48482fa56e5824bbea7b85bbc1 Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Wed, 3 Jul 2019 16:04:40 -0700
Subject: [PATCH] Fix closure annotations for dom-if.

---
 lib/elements/dom-if.js | 41 +++++++++++++++++++++++++----------------
 1 file changed, 25 insertions(+), 16 deletions(-)

diff --git a/lib/elements/dom-if.js b/lib/elements/dom-if.js
index 8dccbc2c33..ec64dc9923 100644
--- a/lib/elements/dom-if.js
+++ b/lib/elements/dom-if.js
@@ -20,25 +20,12 @@ import { fastDomIf, strictTemplatePolicy } from '../utils/settings.js';
 import { showHideChildren } from '../utils/templatize.js';
 
 /**
- * The `<dom-if>` element will stamp a light-dom `<template>` child when
- * the `if` property becomes truthy, and the template can use Polymer
- * data-binding and declarative event features when used in the context of
- * a Polymer element's template.
- *
- * When `if` becomes falsy, the stamped content is hidden but not
- * removed from dom. When `if` subsequently becomes truthy again, the content
- * is simply re-shown. This approach is used due to its favorable performance
- * characteristics: the expense of creating template content is paid only
- * once and lazily.
- *
- * Set the `restamp` property to true to force the stamped content to be
- * created / destroyed when the `if` condition changes.
- *
  * @customElement
  * @polymer
  * @extends PolymerElement
- * @summary Custom element that conditionally stamps and hides or removes
- *   template content based on a boolean flag.
+ * @constructor
+ * @summary Base class for dom-if element; subclassed into concrete
+ *   implementation.
  */
 class DomIfBase extends PolymerElement {
 
@@ -628,6 +615,28 @@ class DomIfLegacy extends DomIfBase {
   }
 }
 
+/**
+ * The `<dom-if>` element will stamp a light-dom `<template>` child when
+ * the `if` property becomes truthy, and the template can use Polymer
+ * data-binding and declarative event features when used in the context of
+ * a Polymer element's template.
+ *
+ * When `if` becomes falsy, the stamped content is hidden but not
+ * removed from dom. When `if` subsequently becomes truthy again, the content
+ * is simply re-shown. This approach is used due to its favorable performance
+ * characteristics: the expense of creating template content is paid only
+ * once and lazily.
+ *
+ * Set the `restamp` property to true to force the stamped content to be
+ * created / destroyed when the `if` condition changes.
+ *
+ * @customElement
+ * @polymer
+ * @extends DomIfBase
+ * @constructor
+ * @summary Custom element that conditionally stamps and hides or removes
+ *   template content based on a boolean flag.
+ */
 export const DomIf = fastDomIf ? DomIfFast : DomIfLegacy;
 
 customElements.define(DomIf.is, DomIf);