Let parser handle factory modifiers.

Let parser handle factory modifiers.

R=johnniwinther@google.com

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

[ahe, 2017-02-27 15:41:49]

ahe@google.com changed reviewers:
+ johnniwinther@google.com

[ahe, 2017-02-27 17:38:20]

Uploaded a new patch set as I had to keep the old code in dart2js :-(

[Johnni Winther, 2017-02-28 13:36:39]

lgtm

https://codereview.chromium.org/2721623002/diff/20001/pkg/front_end/lib/src/f...
File pkg/front_end/lib/src/fasta/kernel/verifier.dart (right):

https://codereview.chromium.org/2721623002/diff/20001/pkg/front_end/lib/src/f...
pkg/front_end/lib/src/fasta/kernel/verifier.dart:44: problem(TreeNode node,
String details) {
No use of this yet?

[ahe, 2017-02-28 13:46:25]

Thank you, Johnni!

https://codereview.chromium.org/2721623002/diff/20001/pkg/front_end/lib/src/f...
File pkg/front_end/lib/src/fasta/kernel/verifier.dart (right):

https://codereview.chromium.org/2721623002/diff/20001/pkg/front_end/lib/src/f...
pkg/front_end/lib/src/fasta/kernel/verifier.dart:44: problem(TreeNode node,
String details) {
On 2017/02/28 13:36:39, Johnni Winther wrote:
> No use of this yet?

It's used a lot in the superclass. Too much, I think. It's kinda accusing me of
generating bad code ;-)

[ahe, 2017-03-01 10:38:08]

Description was changed from

==========
Let parser handle factory modifiers.
==========

to

==========
Let parser handle factory modifiers.

R=johnniwinther@google.com

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

[ahe, 2017-03-01 10:38:09]

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