Enable messages in asserts by default

Enable messages in asserts by default

R=scheglov@google.com

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

[Brian Wilkerson, 2016-12-07 20:21:09]

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

[scheglov, 2016-12-07 20:22:17]

LGTM

[Brian Wilkerson, 2016-12-07 20:23:04]

Description was changed from

==========
Enable messages in asserts by default
==========

to

==========
Enable messages in asserts by default

R=scheglov@google.com

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

[Brian Wilkerson, 2016-12-07 20:23:05]

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

[kevmoo, 2016-12-07 21:09:48]

On 2016/12/07 20:23:05, Brian Wilkerson wrote:
> Committed patchset #1 (id:1) manually as
> e22a2b4626fe11393991f99d554fd52e00bfc0cc (presubmit successful).

Which bug is this?

Want to make sure we're tracking updates to the changelog accordingly, etc

[Brian Wilkerson, 2016-12-07 21:13:49]

> Which bug is this?

There were three issues in the issue tracker that I closed with this change. I
don't have the numbers easily accessible.

> Want to make sure we're tracking updates to the changelog accordingly, etc

No, as usual I forgot to add anything to the change log.

[Lasse Reichstein Nielsen, 2016-12-08 07:00:48]

I think we should just have one entry in the changelog when all platforms are
enabled. Adding an entry just for the analyzer, and the rewriting it later when
dart2js and the VM are enabled, seems like unnecessary work.