Treat NEW as escaping all variables in DeadPropertyAssignmentElimination

Fixes #2924

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=196892725

Author: lauraharker

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

@@ -292,8 +292,8 @@ public void visit(NodeTraversal t, Node n, Node parent) {
         visitAssignmentLhs(n.getFirstChild());
       }
 
-      // Assume that all properties may be read when leaving the function in a CALL or YIELD
-      if (n.isCall() || n.isYield()) {
+      // Assume that all properties may be read when control flow leaves the function
+      if (NodeUtil.isInvocation(n) || n.isYield()) {
         if (ASSUME_CONSTRUCTORS_HAVENT_ESCAPED
             && isConstructor
             && !NodeUtil.referencesThis(n)

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

@@ -4986,6 +4986,13 @@ static boolean isInvocationTarget(Node n) {
         && parent.getFirstChild() == n;
   }
 
+  /**
+   * Returns whether this is a call (including tagged template lits) or new.
+   */
+  static boolean isInvocation(Node n) {
+    return isCallOrNew(n) || n.isTaggedTemplateLit();
+  }
+
   static boolean isCallOrNewArgument(Node n) {
     Node parent = n.getParent();
     return parent != null && isCallOrNew(parent) && parent.getFirstChild() != n;

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

@@ -254,6 +254,28 @@ public void testYield() {
             "}"));
   }
 
+  public void testNew() {
+    // Assume that properties may be read during a constructor call
+    testSame(
+        lines(
+            "var foo = function() {",
+            "  a.b.c = 20;",
+            "  new C;",
+            "  a.b.c = 30;",
+            "}"));
+  }
+
+  public void testTaggedTemplateLit() {
+    // Assume that properties may be read during a tagged template lit invocation
+    testSame(
+        lines(
+            "var foo = function() {",
+            "  a.b.c = 20;",
+            "  doSomething`foo`;",
+            "  a.b.c = 30;",
+            "}"));
+  }
+
   public void testUnknownLookup() {
     testSame(
         lines(