Transitively include outlines of all public members of Class(es).

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

Index: pkg/front_end/lib/src/fasta/kernel/kernel_outline_shaker.dart
diff --git a/pkg/front_end/lib/src/fasta/kernel/kernel_outline_shaker.dart b/pkg/front_end/lib/src/fasta/kernel/kernel_outline_shaker.dart
index ea88a085c9cc5f62d146e48081f74084372e9abb..0b5fc67985587178383e5ec233e0c18a613f75bb 100644
--- a/pkg/front_end/lib/src/fasta/kernel/kernel_outline_shaker.dart
+++ b/pkg/front_end/lib/src/fasta/kernel/kernel_outline_shaker.dart
@@ -100,10 +100,15 @@ class RetainedDataBuilder extends RetainedData {
     markLibrary(cls.parent);
     // TODO(sigmund): retain annotations?
     // visitList(cls.annotations, this);
+    cls.typeParameters.forEach((t) => t.bound.accept(typeMarker));
     markSupertype(cls.supertype);
     markSupertype(cls.mixedInType);
     cls.implementedTypes.forEach(markSupertype);
-    cls.typeParameters.forEach((t) => t.bound.accept(typeMarker));
+    cls.members.forEach((member) {
+      if (!member.name.isPrivate) {
+        markMember(member);

[Paul Berry, 2017/08/30 21:46:45]

I think we should exclude constructors and static members; they can only be
accessed by clients that directly import the library.  E.g. when tree shaking
the API of this file:

class A {
  _B f() { ... }
}
class _B {
  _B() { ... }
  _B.named() { ... }
  g() { ... }
  static h() { ... }
}

We need to include _B.g, since it's accessible via `new A().f().g()`, but we
don't need to include _B's constructors or _B.h.

+      }
+    });
   }
 
   /// Mark the typedef.