Remove WorldImpact from caches when no longer needed.

pkg/compiler/lib/src/dump_info.dart

 // Copyright (c) 2013, 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 dump_info;
 
-import 'dart:convert'
-    show HtmlEscape, JsonEncoder, StringConversionSink, ChunkedConversionSink;
+import 'dart:convert' show
+    ChunkedConversionSink,
+    HtmlEscape,
+    JsonEncoder,
+    StringConversionSink;
 
 import 'package:dart2js_info/info.dart';
 
 import 'common.dart';
-import 'common/tasks.dart' show CompilerTask;
-import 'constants/values.dart' show ConstantValue, InterceptorConstantValue;
-import 'compiler.dart' show Compiler;
+import 'common/tasks.dart' show
+    CompilerTask;
+import 'constants/values.dart' show
+    ConstantValue,
+    InterceptorConstantValue;
+import 'compiler.dart' show
+    Compiler;
+import 'deferred_load.dart' show
+    OutputUnit;
 import 'elements/elements.dart';
 import 'elements/visitor.dart';
-import 'types/types.dart' show TypeMask;
-import 'deferred_load.dart' show OutputUnit;
-import 'js_backend/js_backend.dart' show JavaScriptBackend;
-import 'js_emitter/full_emitter/emitter.dart' as full show Emitter;
+import 'info/send_info.dart' show
+    collectSendMeasurements;
+import 'js_backend/js_backend.dart' show
+    JavaScriptBackend;
+import 'js_emitter/full_emitter/emitter.dart' as full show
+    Emitter;
 import 'js/js.dart' as jsAst;
-import 'universe/universe.dart' show ReceiverConstraint;
-import 'universe/world_impact.dart' show WorldImpact;
-import 'info/send_info.dart' show collectSendMeasurements;
+import 'types/types.dart' show
+    TypeMask;
+import 'universe/universe.dart' show
+    ReceiverConstraint;
+import 'universe/world_impact.dart' show
+    ImpactUse,
+    WorldImpact,
+    WorldImpactVisitorImpl;
 
 class ElementInfoCollector extends BaseElementVisitor<Info, dynamic> {
   final Compiler compiler;
 
   final AllInfo result = new AllInfo();
   final Map<Element, Info> _elementToInfo = <Element, Info>{};
   final Map<ConstantValue, Info> _constantToInfo = <ConstantValue, Info>{};
   final Map<OutputUnit, OutputUnitInfo> _outputToInfo = {};
 
   ElementInfoCollector(this.compiler);
@@ skipping 323 matching lines @@
 /// we can emit them in the dump-info task.
 // TODO(sigmund,het): move more features here. Ideally the dump-info task
 // shouldn't reach into internals of other parts of the compiler. For example,
 // we currently reach into the full emitter and as a result we don't support
 // dump-info when using the startup-emitter (issue #24190).
 abstract class InfoReporter {
   void reportInlined(Element element, Element inlinedFrom);
 }
 
 class DumpInfoTask extends CompilerTask implements InfoReporter {
+  static const ImpactUse IMPACT_USE = const ImpactUse('Dump info');
+
   DumpInfoTask(Compiler compiler) : super(compiler);
 
   String get name => "Dump Info";
 
   ElementInfoCollector infoCollector;
 
   /// The size of the generated output.
   int _programSize;
 
   // A set of javascript AST nodes that we care about the size of.
@@ skipping 32 matching lines @@
   void registerDependency(Element source, Element target) {
     _dependencies.putIfAbsent(source, () => new Set()).add(target);
   }
 
   void registerImpact(Element element, WorldImpact impact) {
     if (compiler.dumpInfo) {
       impacts[element] = impact;
     }
   }
 
+  void unregisterImpact(Element element) {
+    impacts.remove(element);
+  }
+
   /**
    * Returns an iterable of [Selection]s that are used by
    * [element].  Each [Selection] contains an element that is
    * used and the selector that selected the element.
    */
   Iterable<Selection> getRetaining(Element element) {
-    var impact = impacts[element];
+    WorldImpact impact = impacts[element];
     if (impact == null) return const <Selection>[];
 
     var selections = <Selection>[];
-    selections.addAll(impact.dynamicUses.expand((dynamicUse) {
-      return compiler.world.allFunctions
-          .filter(dynamicUse.selector, dynamicUse.mask)
-          .map((e) => new Selection(e, dynamicUse.mask));
-    }));
-    selections.addAll(impact.staticUses
-        .map((staticUse) => new Selection(staticUse.element, null)));
+    compiler.impactStrategy.visitImpact(
+        element,
+        impact,
+        new WorldImpactVisitorImpl(
+            visitDynamicUse: (dynamicUse) {
+              selections.addAll(compiler.world.allFunctions
+                        .filter(dynamicUse.selector, dynamicUse.mask)
+                        .map((e) => new Selection(e, dynamicUse.mask)));
+            },
+            visitStaticUse: (staticUse) {
+              selections.add(new Selection(staticUse.element, null));
+            }
+        ),
+        IMPACT_USE);
     return selections;
   }
 
   // Returns true if we care about tracking the size of
   // this node.
   bool isTracking(jsAst.Node code) {
     if (compiler.dumpInfo) {
       return _tracking.contains(code);
     } else {
       return false;
@@ skipping 125 matching lines @@
 
     ChunkedConversionSink<Object> sink = encoder.startChunkedConversion(
         new StringConversionSink.fromStringSink(buffer));
     sink.add(new AllInfoJsonCodec().encode(result));
     reporter.reportInfo(NO_LOCATION_SPANNABLE, MessageKind.GENERIC, {
       'text': "View the dumped .info.json file at "
           "https://dart-lang.github.io/dump-info-visualizer"
     });
   }
 }