Fixes change tracking in VarCheck.

------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=152940605
google · Apr 12, 2017 · cd1b85d · cd1b85d
1 parent 6c01a3e
commit cd1b85d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/com/google/javascript/jscomp/VarCheck.java b/src/com/google/javascript/jscomp/VarCheck.java
@@ -265,8 +265,9 @@ static void createSynthesizedExternVar(AbstractCompiler compiler, String varName
       nameNode.putBooleanProp(Node.IS_CONSTANT_NAME, true);
     }
 
-    getSynthesizedExternsRoot(compiler).addChildToBack(IR.var(nameNode));
-    compiler.reportCodeChange();
+    Node syntheticExternVar = IR.var(nameNode);
+    getSynthesizedExternsRoot(compiler).addChildToBack(syntheticExternVar);
+    compiler.reportChangeToEnclosingScope(syntheticExternVar);
   }
 
   /**

diff --git a/test/com/google/javascript/jscomp/VarCheckTest.java b/test/com/google/javascript/jscomp/VarCheckTest.java
@@ -42,7 +42,6 @@ protected void setUp() throws Exception {
     super.setUp();
     // Setup value set by individual tests to the appropriate defaults.
     super.allowExternsChanges(true);
-    validateAstChangeMarking(false);
     strictModuleDepErrorLevel = CheckLevel.OFF;
     externValidationErrorLevel = null;
     sanityCheck = false;