Do not sort named parameters on FunctionNode.

Do not sort named parameters on FunctionNode.

This was originally done to be consistent with FunctionType,
where the named parameters are sorted, but for FunctionNode
it is not necessary.

It causes issues for natives that expect parameters to be
declared in a certain order. It is also an issue if we
ever want to reuse the kernel format for summaries, where
it is also preferable to present the named parameters in
their original order.

BUG=
R=kmillikin@google.com

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

[asgerf, 2016-11-24 13:28:08]

Description was changed from

==========
Do not sort named parameters on FunctionNode.

BUG=
==========

to

==========
Do not sort named parameters on FunctionNode.

This was originally done to be consistent with FunctionType,
where the named parameters are sorted.

It causes issues for natives that expect parameters to be
declared in a certain order. It is also an issue if we
ever want to reuse the kernel format for summaries, where
it is also preferable to present the named parameters in
their original order.

BUG=
==========

[asgerf, 2016-11-24 13:28:22]

Description was changed from

==========
Do not sort named parameters on FunctionNode.

This was originally done to be consistent with FunctionType,
where the named parameters are sorted.

It causes issues for natives that expect parameters to be
declared in a certain order. It is also an issue if we
ever want to reuse the kernel format for summaries, where
it is also preferable to present the named parameters in
their original order.

BUG=
==========

to

==========
Do not sort named parameters on FunctionNode.

This was originally done to be consistent with FunctionType,
where the named parameters are sorted, but for FunctionNode
it is not necessary.

It causes issues for natives that expect parameters to be
declared in a certain order. It is also an issue if we
ever want to reuse the kernel format for summaries, where
it is also preferable to present the named parameters in
their original order.

BUG=
==========

[asgerf, 2016-11-24 13:28:54]

asgerf@google.com changed reviewers:
+ kmillikin@google.com

[Kevin Millikin (Google), 2016-11-25 09:51:19]

LGTM.

[asgerf, 2016-11-25 10:22:41]

Description was changed from

==========
Do not sort named parameters on FunctionNode.

This was originally done to be consistent with FunctionType,
where the named parameters are sorted, but for FunctionNode
it is not necessary.

It causes issues for natives that expect parameters to be
declared in a certain order. It is also an issue if we
ever want to reuse the kernel format for summaries, where
it is also preferable to present the named parameters in
their original order.

BUG=
==========

to

==========
Do not sort named parameters on FunctionNode.

This was originally done to be consistent with FunctionType,
where the named parameters are sorted, but for FunctionNode
it is not necessary.

It causes issues for natives that expect parameters to be
declared in a certain order. It is also an issue if we
ever want to reuse the kernel format for summaries, where
it is also preferable to present the named parameters in
their original order.

BUG=
R=kmillikin@google.com

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

[asgerf, 2016-11-25 10:22:43]

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