Rename ClosureClassMaps to ClosureDataLookup

pkg/compiler/lib/src/compiler.dart

Index: pkg/compiler/lib/src/compiler.dart
diff --git a/pkg/compiler/lib/src/compiler.dart b/pkg/compiler/lib/src/compiler.dart
index aa3a5ba692f2c0be69316011b6ef91449c0c6465..b591b4042bd5aa8550d3c77d7260fa1742475abc 100644
--- a/pkg/compiler/lib/src/compiler.dart
+++ b/pkg/compiler/lib/src/compiler.dart
@@ -144,7 +144,7 @@ abstract class Compiler {
   LibraryLoaderTask libraryLoader;
   SerializationTask serialization;
   ResolverTask resolver;
-  closureMapping.ClosureTask closureToClassMapper;
+  closureMapping.ClosureTask closureDataLookup;

[Siggi Cherem (dart-lang), 2017/06/14 23:17:12]

not sure if there are some dependencies that still use other APIs ClosureTask,
but if not, it might be nice to also change the type here to be
ClosureDataLookup<Node> instead of ClosureTask.

If this pulls more stuff in, no worries, we'll get to it later.

[Emily Fortuna, 2017/06/14 23:44:13]

Done.

[Emily Fortuna, 2017/06/14 23:54:12]

oh, and update to this -- this requires more changes because
element_strategy.dart calls "createClosureClasses" which is on ClosureTask but
not ClosureDataLookup. However, I was noticing as I've been working on the next
CL that this existing layout should change. So, expect that in a followup CL.

   TypeCheckerTask checker;
   GlobalTypeInferenceTask globalInference;
   JavaScriptBackend backend;
@@ -226,7 +226,7 @@ abstract class Compiler {
           measurer),
       parser = new ParserTask(this),
       resolver = createResolverTask(),
-      closureToClassMapper = new closureMapping.ClosureTask(this),
+      closureDataLookup = new closureMapping.ClosureTask(this),
       checker = new TypeCheckerTask(this),
       globalInference = new GlobalTypeInferenceTask(this),
       constants = backend.constantCompilerTask,