Rollback of "Move type removal to happen just after ambiguation"

blickly · copybara-github · commit 4f21a1dc53ec · 2020-07-17T14:03:15.000-07:00
PiperOrigin-RevId: 321849337
diff --git a/src/com/google/javascript/jscomp/DefaultPassConfig.java b/src/com/google/javascript/jscomp/DefaultPassConfig.java
@@ -799,8 +799,6 @@ protected List<PassFactory> getOptimizations() {
       passes.add(ambiguateProperties);
     }
 
-    passes.add(removeTypes);
-
     if (options.propertyRenaming == PropertyRenamingPolicy.ALL_UNQUOTED) {
       passes.add(renameProperties);
     }
@@ -873,6 +871,8 @@ protected List<PassFactory> getOptimizations() {
       passes.add(latePeepholeOptimizations);
     }
 
+    passes.add(removeTypes);
+
     if (options.anonymousFunctionNaming == AnonymousFunctionNamingPolicy.UNMAPPED) {
       passes.add(nameUnmappedAnonymousFunctions);
     }

Original file line number	Diff line number	Diff line change
`@@ -799,8 +799,6 @@ protected List<PassFactory> getOptimizations() {`
`799`	`799`	`passes.add(ambiguateProperties);`
`800`	`800`	`}`
`801`	`801`
`802`		`- passes.add(removeTypes);`
`803`		`-`
`804`	`802`	`if (options.propertyRenaming == PropertyRenamingPolicy.ALL_UNQUOTED) {`
`805`	`803`	`passes.add(renameProperties);`
`806`	`804`	`}`
`@@ -873,6 +871,8 @@ protected List<PassFactory> getOptimizations() {`
`873`	`871`	`passes.add(latePeepholeOptimizations);`
`874`	`872`	`}`
`875`	`873`
	`874`	`+ passes.add(removeTypes);`
	`875`	`+`
`876`	`876`	`if (options.anonymousFunctionNaming == AnonymousFunctionNamingPolicy.UNMAPPED) {`
`877`	`877`	`passes.add(nameUnmappedAnonymousFunctions);`
`878`	`878`	`}`