File additional bugs against TypeScript and Polymer for the workarounds

in transformer mode.

Closes #529

Author: tbosch

src/transformer_util.ts

@@ -332,6 +332,8 @@ function resetNodeTextRangeToPreventDuplicateComments<T extends ts.Node>(node: T
  * Reads in the leading comment text ranges of the given node,
  * converts them into `ts.SyntheticComment`s and stores them on the node.
  *
+ * Note: This would be greatly simplified with https://github.com/Microsoft/TypeScript/issues/17615.
+ *
  * @param lastCommentEnd The end of the last comment
  * @return The end of the last found comment, -1 if no comment was found.
  */
@@ -357,6 +359,8 @@ function synthesizeLeadingComments(
  * Reads in the trailing comment text ranges of the given node,
  * converts them into `ts.SyntheticComment`s and stores them on the node.
  *
+ * Note: This would be greatly simplified with https://github.com/Microsoft/TypeScript/issues/17615.
+ *
  * @return The end of the last found comment, -1 if no comment was found.
  */
 function synthesizeTrailingComments(sourceFile: ts.SourceFile, node: ts.Node): number {
@@ -418,6 +422,8 @@ function visitNodeStatementsWithSynthesizedComments<T extends ts.Node>(
  *
  * A Detached leading comment is the first comment in a SourceFile / Block
  * that is separated with a newline from the first statement.
+ *
+ * Note: This would be greatly simplified with https://github.com/Microsoft/TypeScript/issues/17615.
  */
 function synthesizeDetachedLeadingComments(
     sourceFile: ts.SourceFile, node: ts.Node, statements: ts.NodeArray<ts.Statement>):
@@ -445,6 +451,8 @@ function synthesizeDetachedLeadingComments(
  *
  * A Detached trailing comment are all comments after the first newline
  * the follows the last statement in a SourceFile / Block.
+ *
+ * Note: This would be greatly simplified with https://github.com/Microsoft/TypeScript/issues/17615.
  */
 function synthesizeDetachedTrailingComments(
     sourceFile: ts.SourceFile, node: ts.Node, statements: ts.NodeArray<ts.Statement>):

src/tsickle.ts

@@ -718,8 +718,7 @@ class Annotator extends ClosureRewriter {
         // for annotating binding patterns.  See issue #128.
         // Don't emit type annotation when the variable statement is a @polymerBehavior,
         // as otherwise the polymer closure checker will fail.
-        // TODO(tbosch): file an issue with Polymer, tracked in
-        // https://github.com/angular/tsickle/issues/529.
+        // See b/64389806
         if (this.polymerBehaviorStackCount === 0 &&
             varDecl.name.kind === ts.SyntaxKind.Identifier) {
           this.emitJSDocType(varDecl);
@@ -790,11 +789,10 @@ class Annotator extends ClosureRewriter {
           // Don't emit type casts for Polymer behaviors that are declared
           // by calling the Polymer function
           // as the Polymer closure plugin does not work when emitting them.
+          // See b/64389806.
           // Note: This only matters in the transformer version of tsickle,
           // as the non transformer version never emitted type casts due to
           // https://github.com/Microsoft/TypeScript/issues/9873 (see below).
-          // TODO(tbosch): file an issue with Polymer, tracked in
-          // https://github.com/angular/tsickle/issues/529.
           return false;
         }
         // When using a type casts in template expressions,