Split implementation of KernelToElementMapImpl

pkg/compiler/lib/src/js_model/js_strategy.dart

Index: pkg/compiler/lib/src/js_model/js_strategy.dart
diff --git a/pkg/compiler/lib/src/js_model/js_strategy.dart b/pkg/compiler/lib/src/js_model/js_strategy.dart
index e2279f667207b8e029c3634e5b8f9665c8ca3302..ad2679edabdc73c53be5d70395a1d5785045a850 100644
--- a/pkg/compiler/lib/src/js_model/js_strategy.dart
+++ b/pkg/compiler/lib/src/js_model/js_strategy.dart
@@ -40,13 +40,14 @@ class JsBackendStrategy implements KernelBackendStrategy {
   KernelToElementMapForBuilding _elementMap;
   ClosureConversionTask _closureDataLookup;
   final GlobalLocalsMap _globalLocalsMap = new GlobalLocalsMap();
+  Sorter _sorter;
 
   JsBackendStrategy(this._compiler);
 
   KernelToElementMapForBuilding get elementMap {
     if (_elementMap == null) {
       KernelFrontEndStrategy strategy = _compiler.frontendStrategy;
-      KernelToElementMapForBuilding elementMap = strategy.elementMap;
+      KernelToElementMapForImpact elementMap = strategy.elementMap;

[Siggi Cherem (dart-lang), 2017/06/30 22:19:28]

seems like we have a misnomer then?

[Johnni Winther, 2017/07/03 08:00:57]

No. The elementMap of the frontend strategy is a KernelToElementMapForImpact
whereas the elementMap of the backend strategy is a
KernelToElementMapForBuilding.

Changed the type of `KernelFrontEndStrategy.elementMap` to
[KernelToElementMapForImpact] instead of [KernelToElementMapForImpactImpl].

       _elementMap = new JsKernelToElementMap(
           _compiler.reporter,
           _compiler.environment,
@@ -133,7 +134,7 @@ class JsBackendStrategy implements KernelBackendStrategy {
 
   @override
   Sorter get sorter {
-    throw new UnimplementedError('JsBackendStrategy.sorter');
+    return _sorter ??= new KernelSorter(elementMap);
   }
 
   @override