Rollback of "Change InlineAliases from an always-on pass to a CommonJ…

…S/J2cl-only pass." This change is actually a no-op until J2clPass defaults to off, which will be submitted separately. Then this change can be rolled forward.

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

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

@@ -397,7 +397,7 @@ protected List<PassFactory> getChecks() {
       checks.add(newTypeInference);
     }
 
-    if (options.processCommonJSModules || options.j2clPassMode.shouldAddJ2clPasses()) {
+    if (!options.allowsHotswapReplaceScript()) {
       checks.add(inlineTypeAliases);
     }
 

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

@@ -21,6 +21,7 @@
 import com.google.javascript.rhino.JSDocInfo;
 import com.google.javascript.rhino.JSDocInfo.Visibility;
 import com.google.javascript.rhino.Node;
+
 import java.util.LinkedHashMap;
 import java.util.LinkedHashSet;
 import java.util.Map;
@@ -48,24 +49,11 @@ final class InlineAliases implements CompilerPass {
     this.compiler = compiler;
   }
 
-  private boolean shouldInlineAliases() {
-    if (compiler.getOptions().processCommonJSModules) {
-      // TODO(blickly): Remove this case once CommonJS modules are rewritten like goog.modules.
-      return true;
-    }
-    if (J2clSourceFileChecker.shouldSkipExecution(compiler)) {
-      return false;
-    }
-    return true;
-  }
-
   @Override
   public void process(Node externs, Node root) {
-    if (shouldInlineAliases()) {
-      namespace = new GlobalNamespace(compiler, root);
-      NodeTraversal.traverseEs6(compiler, root, new AliasesCollector());
-      NodeTraversal.traverseEs6(compiler, root, new AliasesInliner());
-    }
+    namespace = new GlobalNamespace(compiler, root);
+    NodeTraversal.traverseEs6(compiler, root, new AliasesCollector());
+    NodeTraversal.traverseEs6(compiler, root, new AliasesInliner());
   }
 
   private class AliasesCollector extends AbstractPostOrderCallback {