Ast changes to support the covariant keyword

Ast changes to support the covariant keyword

R=paulberry@google.com

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

[Brian Wilkerson, 2017-01-16 23:25:21]

brianwilkerson@google.com changed reviewers:
+ paulberry@google.com

[Brian Wilkerson, 2017-01-16 23:25:21]

I didn't update either the constructors or the factory methods. Not doing so
seems better for backporting to the 0.29 version, but I can do so if you think
it's better for the bleeding edge version.

[Paul Berry, 2017-01-17 13:28:00]

lgtm

[Brian Wilkerson, 2017-01-17 14:44:48]

Description was changed from

==========
Ast changes to support the covariant keyword
==========

to

==========
Ast changes to support the covariant keyword

R=paulberry@google.com

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

[Brian Wilkerson, 2017-01-17 14:44:49]

Committed patchset #1 (id:1) manually as
38264a14630a4833c9b5c4bc3ce8949d6cfe395b (presubmit successful).

[scheglov, 2017-01-18 04:29:58]

scheglov@google.com changed reviewers:
+ scheglov@google.com

[scheglov, 2017-01-18 04:29:59]

https://codereview.chromium.org/2637753003/diff/1/pkg/analyzer/lib/src/dart/a...
File pkg/analyzer/lib/src/dart/ast/ast.dart (right):

https://codereview.chromium.org/2637753003/diff/1/pkg/analyzer/lib/src/dart/a...
pkg/analyzer/lib/src/dart/ast/ast.dart:8181: result.add(covariantKeyword);
Do we need to update beginToken?

[Brian Wilkerson, 2017-01-18 17:19:30]

https://codereview.chromium.org/2637753003/diff/1/pkg/analyzer/lib/src/dart/a...
File pkg/analyzer/lib/src/dart/ast/ast.dart (right):

https://codereview.chromium.org/2637753003/diff/1/pkg/analyzer/lib/src/dart/a...
pkg/analyzer/lib/src/dart/ast/ast.dart:8181: result.add(covariantKeyword);
Yes. Good catch. I'll have an update out shortly.