Migrate UnlinkedParam.defaultValue to UnlinkedExecutable.bodyExpr.

Migrate UnlinkedParam.defaultValue to UnlinkedExecutable.bodyExpr.

To do AST-based type inference involving function closures, we have
added a `bodyExpr` field to UnlinkedExecutable.  Since every
UnlinkedParam with a default value has a corresponding
UnlinkedExecutable representing its initializer, it makes sense to
store the constant expression there for uniformity.

R=scheglov@google.com

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

[Paul Berry, 2016-05-26 11:33:21]

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

[scheglov, 2016-05-26 13:50:20]

LGTM

[Paul Berry, 2016-05-26 15:12:43]

Description was changed from

==========
Migrate UnlinkedParam.defaultValue to UnlinkedExecutable.bodyExpr.

To do AST-based type inference involving function closures, we have
added a `bodyExpr` field to UnlinkedExecutable.  Since every
UnlinkedParam with a default value has a corresponding
UnlinkedExecutable representing its initializer, it makes sense to
store the constant expression there for uniformity.
==========

to

==========
Migrate UnlinkedParam.defaultValue to UnlinkedExecutable.bodyExpr.

To do AST-based type inference involving function closures, we have
added a `bodyExpr` field to UnlinkedExecutable.  Since every
UnlinkedParam with a default value has a corresponding
UnlinkedExecutable representing its initializer, it makes sense to
store the constant expression there for uniformity.

R=scheglov@google.com

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

[Paul Berry, 2016-05-26 15:12:44]

Committed patchset #1 (id:1) manually as
8d43e021744dc880ed3f9dc04145d87765b9a9ec (presubmit successful).