Fix RewriteClassMembers to move the jsdoc on instance field when it is moved to the constructor.

PiperOrigin-RevId: 620291707

Author: Closure Team

src/com/google/javascript/jscomp/RewriteClassMembers.java

@@ -341,6 +341,12 @@ private Node convNonCompFieldToGetProp(Node receiver, Node noncomputedField) {
         (fieldValue != null)
             ? astFactory.createAssignStatement(getProp, fieldValue.detach())
             : astFactory.exprResult(getProp);
+    // Move any JSDoc from the field declaration to the child of the EXPR_RESULT,
+    // which represents the new declaration.
+    // noncomputedField is already detached, so there's no benefit to calling
+    // NodeUtil.getBestJSDocInfo(noncomputedField). For now at least,
+    // the JSDocInfo we want will always be directly on noncomputedField in all cases.
+    result.getFirstChild().setJSDocInfo(noncomputedField.getJSDocInfo());
     result.srcrefTreeIfMissing(noncomputedField);
     return result;
   }

test/com/google/javascript/jscomp/RewriteClassMembersTest.java

@@ -283,12 +283,14 @@ public void testThisInNonStaticPublicField() {
     test(
         lines(
             "class A {", //
+            "  /** @suppress {partialAlias} */",
             "  b = 'word';",
             "  c = this.b;",
             "}"),
         lines(
             "class A {",
             "  constructor() {",
+            "    /** @suppress {partialAlias} */",
             "    this.b = 'word';",
             "    this.c = this.b;",
             "  }",