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

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

R=ahe@google.com, paulberry@google.com, sigmund@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/39b921ecce35656f4e71844a9b1b73eddbbf6947

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

https://codereview.chromium.org/3004943002/diff/1/pkg/front_end/lib/src/fasta...
File pkg/front_end/lib/src/fasta/kernel/kernel_outline_shaker.dart (right):

https://codereview.chromium.org/3004943002/diff/1/pkg/front_end/lib/src/fasta...
pkg/front_end/lib/src/fasta/kernel/kernel_outline_shaker.dart:109:
markMember(member);
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.

[scheglov, 2017-08-31 04:28:10]

PTAL

[Paul Berry, 2017-08-31 15:43:49]

lgtm

[scheglov, 2017-08-31 18:35:12]

Description was changed from

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

R=ahe@google.com, paulberry@google.com, sigmund@google.com
BUG=
==========

to

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

R=ahe@google.com, paulberry@google.com, sigmund@google.com
BUG=

Committed:
https://github.com/dart-lang/sdk/commit/39b921ecce35656f4e71844a9b1b73eddbbf6947
==========

[scheglov, 2017-08-31 18:35:14]

Committed patchset #2 (id:20001) manually as
39b921ecce35656f4e71844a9b1b73eddbbf6947 (presubmit successful).