Access NativeData only through ClosedWorld

pkg/compiler/lib/src/js_emitter/code_emitter_task.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.
 
 library dart2js.js_emitter.code_emitter_task;
 
 import 'package:js_runtime/shared/embedded_names.dart' show JsBuiltin;
 
 import '../common.dart';
 import '../common/tasks.dart' show CompilerTask;
@@ skipping 18 matching lines @@
 const USE_LAZY_EMITTER = const bool.fromEnvironment("dart2js.use.lazy.emitter");
 
 /**
  * Generates the code for all used classes in the program. Static fields (even
  * in classes) are ignored, since they can be treated as non-class elements.
  *
  * The code for the containing (used) methods must exist in the `universe`.
  */
 class CodeEmitterTask extends CompilerTask {
   TypeTestRegistry typeTestRegistry;
-  NativeEmitter nativeEmitter;
+  NativeEmitter _nativeEmitter;
   MetadataCollector metadataCollector;
   EmitterFactory _emitterFactory;
   Emitter _emitter;
   final Compiler compiler;
 
   JavaScriptBackend get backend => compiler.backend;
 
   @deprecated
   // This field should be removed. It's currently only needed for dump-info and
   // tests.
   // The field is set after the program has been emitted.
   /// Contains a list of all classes that are emitted.
   Set<ClassEntity> neededClasses;
 
   CodeEmitterTask(
       Compiler compiler, bool generateSourceMap, bool useStartupEmitter)
       : compiler = compiler,
         super(compiler.measurer) {
-    nativeEmitter = new NativeEmitter(this);
     if (USE_LAZY_EMITTER) {
       _emitterFactory = new lazy_js_emitter.EmitterFactory();
     } else if (useStartupEmitter) {
       _emitterFactory = new startup_js_emitter.EmitterFactory(
           generateSourceMap: generateSourceMap);
     } else {
       _emitterFactory = new full_js_emitter.EmitterFactory(
           generateSourceMap: generateSourceMap);
     }
   }
 
+  NativeEmitter get nativeEmitter {
+    assert(invariant(NO_LOCATION_SPANNABLE, _nativeEmitter != null,
+        message: "NativeEmitter has not been created yet."));
+    return _nativeEmitter;
+  }
+
   Emitter get emitter {
     assert(invariant(NO_LOCATION_SPANNABLE, _emitter != null,
         message: "Emitter has not been created yet."));
     return _emitter;
   }
 
   /// Returns the [Sorter] use for ordering elements in the generated
   /// JavaScript.
   // TODO(johnniwinther): Switch this based on the used entity model.
   Sorter get sorter => const ElementSorter();
@@ skipping 76 matching lines @@
     typeTestRegistry.computeRequiredTypeChecks(backend.rtiChecksBuilder);
     // Compute the classes needed by RTI.
     typeTestRegistry.computeRtiNeededClasses(backend.rtiSubstitutions,
         backend.mirrorsData, backend.generatedCode.keys);
   }
 
   /// Creates the [Emitter] for this task.
   void createEmitter(Namer namer, ClosedWorld closedWorld,
       CodegenWorldBuilder codegenWorldBuilder) {
     measure(() {
+      _nativeEmitter = new NativeEmitter(this, closedWorld.nativeData);
       _emitter = _emitterFactory.createEmitter(this, namer, closedWorld);
       metadataCollector = new MetadataCollector(
           compiler.options,
           compiler.reporter,
           compiler.deferredLoadTask,
           _emitter,
           backend.constants,
           backend.typeVariableCodegenAnalysis,
           backend.mirrorsData,
           backend.rtiEncoder);
       typeTestRegistry = new TypeTestRegistry(codegenWorldBuilder, closedWorld);
     });
   }
 
   int assembleProgram(Namer namer, ClosedWorld closedWorld) {
     return measure(() {
       _finalizeRti();
       ProgramBuilder programBuilder = new ProgramBuilder(
           compiler.options,
           compiler.elementEnvironment,
           compiler.commonElements,
           compiler.types,
           compiler.deferredLoadTask,
           compiler.closureToClassMapper,
           compiler.codegenWorldBuilder,
           backend.nativeCodegenEnqueuer,
           backend.backendUsage,
           backend.constants,
-          backend.nativeData,
+          closedWorld.nativeData,
           backend.rtiNeed,
           backend.mirrorsData,
           backend.interceptorData,
           backend.superMemberData,
           typeTestRegistry.rtiChecks,
           backend.rtiEncoder,
           backend.rtiSubstitutions,
           backend.jsInteropAnalysis,
           backend.oneShotInterceptorData,
           backend.customElementsCodegenAnalysis,
@@ skipping 65 matching lines @@
 
   /// Returns the JS code for accessing the given [constant].
   jsAst.Expression constantReference(ConstantValue constant);
 
   /// Returns the JS template for the given [builtin].
   jsAst.Template templateForBuiltin(JsBuiltin builtin);
 
   /// Returns the size of the code generated for a given output [unit].
   int generatedSize(OutputUnit unit);
 }