Revert "Tweak public APIs and use them in patch_sdk, dart2js, and kernel-service."

pkg/front_end/lib/src/fasta/kernel/kernel_outline_shaker.dart

 // Copyright (c) 2016, 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.
 
 /// A transformation to create a self-contained modular kernel without
 /// unnecessary references to other libraries.
 library fasta.kernel.kernel_outline_shaker;
 
 import 'package:kernel/ast.dart';
 import 'package:kernel/core_types.dart';
 
 import '../errors.dart' show internalError;
 
 /// Removes unnecessary libraries, classes, and members from [program].
 ///
 /// This applies a simple "tree-shaking" technique: the full body of libraries
 /// whose URI match [isIncluded] is preserved, and so is the outline of the
-/// members and classes which are transitively visible from the
-/// included libraries.
+/// members and classes which are indicated by [data] (which should
+/// practically include all members and classes transitively visible from the
+/// included libraries).
 ///
 /// The intent is that the resulting program has the entire code that is meant
 /// to be included and the minimum required to prevent dangling references and
 /// allow modular program transformations.
 ///
 /// Note that the resulting program may include libraries not in [isIncluded],
 /// but those will be marked as external. There should be no method bodies for
 /// any members of those libraries.
-void trimProgram(Program program, bool isIncluded(Uri uri)) {
-  var data = new RetainedDataBuilder();
-  new RootsMarker(new CoreTypes(program), data).run(program, isIncluded);
+void trimProgram(Program program, RetainedData data, bool isIncluded(Uri uri)) {
   new KernelOutlineShaker(data, isIncluded).transform(program);
 }
 
 /// Informs about which libraries, classes, and members should be retained by
 /// the [KernelOutlineShaker] when tree-shaking.
 abstract class RetainedData {
   /// Whether a library should be preserved and mark as external.
   bool isLibraryUsed(Library library);
 
   /// Whether a class should be preserved. If a class is preserved, its
@@ skipping 355 matching lines @@
   /// Types appear to be encoded directly, so we have no need to preserve
   /// typedefs.
   // TODO(sigmund): revisit if this is not the case, the `inputError` in
   // [RootsMarker] is meant to detect this.
   Typedef visitTypedef(Typedef node) => null;
 
   TreeNode defaultTreeNode(TreeNode node) => node;
 }
 
 typedef bool Filter(Uri uri);