Track emitted classes in incremental compiler.

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

Index: dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart
diff --git a/dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart b/dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart
index de0592bf213c3ee21daf25628e90cbb644ad198d..e8a991006eede407d83704e861f004971c3f3a75 100644
--- a/dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart
+++ b/dart/pkg/compiler/lib/src/js_emitter/code_emitter_task.dart
@@ -321,21 +321,24 @@ class CodeEmitterTask extends CompilerTask {
 
     backend.generatedCode.keys.forEach(addSurroundingLibraryToSet);
     neededClasses.forEach(addSurroundingLibraryToSet);
-}
+  }
+
+  void computeNeeds() {

[karlklose, 2014/12/01 10:13:31]

Perhaps you can find a better name than "needs". How about
"computeEntitiesToEmit"?

At least the, function should have a comment that describes what it is supposed
to compute.

[ahe, 2014/12/01 10:28:35]

Since most of the methods are called "computedNeededFoo", I think
computeNeed...something is a good fit.

How about: computeNeededEntities?
I'd rather not add documentation as I'm not 100% sure exactly what is computed.
Or I can add something vague like:

/// Initializes various collections that inform the emitter about what is
needed.

The reason why I'd rather not add documentation is that I'm not 100% sure that
everything that I think should be computed by this method is actually computed
by this method.

WDYT?

[ahe, 2014/12/01 13:58:52]

Discussed offline: I've renamed the method to computeAllNeededEntities.

+    // Compute the required type checks to know which classes need a
+    // 'is$' method.
+    typeTestEmitter.computeRequiredTypeChecks();
+
+    computeNeededDeclarations();
+    computeNeededConstants();
+    computeNeededStatics();
+    computeNeededLibraries();
+  }
 
   void assembleProgram() {
     measure(() {
       emitter.invalidateCaches();
 
-      // Compute the required type checks to know which classes need a
-      // 'is$' method.
-      typeTestEmitter.computeRequiredTypeChecks();
-
-      computeNeededDeclarations();
-      computeNeededConstants();
-      computeNeededStatics();
-      computeNeededLibraries();
-
+      computeNeeds();
 
       Program program;
       if (USE_NEW_EMITTER) {