Skip constructing global namespace and converting super constructor calls when unnecessary during class rewriting

Currently, the global namespace construction and es6ConvertSuperConstructorCalls both get run even if the SCRIPT does not have Feature.SUPER specifically. This CL fixes it to only run if the SCRIPT has a Feature.SUPER.

PiperOrigin-RevId: 621842089

Author: rishipal

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

@@ -25,6 +25,7 @@
 import com.google.javascript.jscomp.GlobalNamespace.Ref;
 import com.google.javascript.jscomp.colors.Color;
 import com.google.javascript.jscomp.colors.StandardColors;
+import com.google.javascript.jscomp.parsing.parser.FeatureSet.Feature;
 import com.google.javascript.rhino.IR;
 import com.google.javascript.rhino.Node;
 import com.google.javascript.rhino.QualifiedName;
@@ -71,6 +72,10 @@ public Es6ConvertSuperConstructorCalls(AbstractCompiler compiler) {
 
   @Override
   public boolean shouldTraverse(NodeTraversal t, Node n, Node parent) {
+    if (n.isScript() && !NodeUtil.getFeatureSetOfScript(n).contains(Feature.SUPER)) {
+      // If a script contains Feature.SUPER, only then process super constructor calls for it
+      return false;
+    }
     if (n.isFunction()) {
       // TODO(bradfordcsmith): Avoid creating data for non-constructor functions.
       constructorDataStack.push(new ConstructorData(n));

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

@@ -71,8 +71,10 @@ public void process(Node externs, Node root) {
     // have an invalid AST state between passes.
     // TODO(bradfordcsmith): It would probably be more readable and efficient to merge the super
     //     constructor rewriting logic into this class.
+    // The code here only creates the GlobalNamespace object which is very cheap. The expensive
+    // building of global namespace happens inside es6ConvertSuperConstructorCalls pass.
     convertSuperConstructorCalls.setGlobalNamespace(new GlobalNamespace(compiler, externs, root));
-    TranspilationPasses.processTranspile(compiler, root, features, convertSuperConstructorCalls);
+    NodeTraversal.traverse(compiler, root, convertSuperConstructorCalls);
     TranspilationPasses.maybeMarkFeaturesAsTranspiledAway(compiler, root, features);
   }
 
@@ -256,7 +258,9 @@ private Node createObjectDotDefineProperty() {
         this.transpilationNamespace, "Object.defineProperty");
   }
 
-  /** @param member A getter or setter */
+  /**
+   * @param member A getter or setter
+   */
   private void addToDefinePropertiesObject(ClassDeclarationMetadata metadata, Node member) {
     Preconditions.checkArgument(!member.isComputedProp());
     Node obj =