dart2js: Only emit constructors of the native class interceptors that are used.

pkg/compiler/lib/src/js_emitter/native_emitter.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 part of dart2js.js_emitter;
 
 class NativeEmitter {
 
   final Map<Element, ClassBuilder> cachedBuilders;
 
@@ skipping 214 matching lines @@
       generateClassInfo(backend.jsInterceptorClass);
       for (ClassElement classElement in classes) {
         generateClassInfo(classElement);
       }
     }
 
     // Emit the native class interceptors that were actually used.
     for (ClassElement classElement in classes) {
       if (!classElement.isNative) continue;
       if (neededClasses.contains(classElement)) {
+        ClassBuilder builder = builders[classElement];
+
+        // In CSP mode [emitClassConstructor] and [emitClassGettersSetters] have
+        // a side-effect on "precompiled" functions in [OldEmitter]. For this
+        // reason, it is important that we don't call these methods before we
+        // are certain that a class is needed.
+
+        emitterTask.oldEmitter.classEmitter
+            .emitConstructorsForCSP(classElement);
+
+        // [emitClassGettersSettersForCSP] does not affect whether or not a
+        // class is needed. If getters/setters are emitted, the class has fields
+        // and is therefore non-trivial.
+        emitterTask.oldEmitter.classEmitter.emitClassGettersSettersForCSP(
+            classElement, builder);
+
         // Define interceptor class for [classElement].
         emitterTask.oldEmitter.classEmitter.emitClassBuilderWithReflectionData(
             backend.namer.getNameOfClass(classElement),
             classElement, builders[classElement],
             emitterTask.oldEmitter.getElementDescriptor(classElement));
         emitterTask.oldEmitter.needsClassSupport = true;
       }
     }
   }
 
@@ skipping 49 matching lines @@
     if (compiler.hasIncrementalSupport) {
       builder = cachedBuilders[classElement];
       if (builder != null) return builder;
       builder = new ClassBuilder(classElement, backend.namer);
       cachedBuilders[classElement] = builder;
     } else {
       builder = new ClassBuilder(classElement, backend.namer);
     }
     builder.superName = superName;
 
-    emitterTask.oldEmitter.classEmitter.emitClassConstructor(
-        classElement, builder);
     bool hasFields = emitterTask.oldEmitter.classEmitter.emitFields(
         classElement, builder, classIsNative: true);
     int propertyCount = builder.properties.length;
-    emitterTask.oldEmitter.classEmitter.emitClassGettersSetters(
+    emitterTask.oldEmitter.classEmitter.emitCheckedClassSetters(
         classElement, builder);
     emitterTask.oldEmitter.classEmitter.emitInstanceMembers(
         classElement, builder);
     emitterTask.oldEmitter.typeTestEmitter.emitIsTests(classElement, builder);
 
     if (!hasFields &&
         builder.properties.length == propertyCount &&
         superclass is! MixinApplicationElement) {
       builder.isTrivial = true;
     }
@@ skipping 150 matching lines @@
         targetOutput.add(';');
       }
       targetOutput.addBuffer(jsAst.prettyPrint(
           new jsAst.ExpressionStatement(init), compiler));
       targetOutput.add('\n');
     }
 
     targetOutput.add('\n');
   }
 }