Allowing `e` of any type in `void f(...) => e` in the analyzer.

Allowing `e` of any type in `void f(...) => e` in the analyzer.

Re-land, now including adjustment of two tests in pkg/analyzer/test.

R=brianwilkerson@google.com

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

[eernst, 2017-05-10 12:13:54]

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

[eernst, 2017-05-10 12:13:55]

This is a re-landing of CL 2870063002, which caused 3 tests in pkg/analyzer/test
to fail. This CL adjusts those tests such that they expect the new analyzer
behavior.

go/darto says 'Tree is closed (Dartium 50..)'. If the tree is opened again soon
then I'd like to land it, so this CL is "possibly TBR".

[Brian Wilkerson, 2017-05-10 15:44:17]

lgtm

[eernst, 2017-05-11 08:10:22]

Description was changed from

==========
Allowing `e` of any type in `void f(...) => e` in the analyzer.

Re-land, now including adjustment of two tests in pkg/analyzer/test.
==========

to

==========
Allowing `e` of any type in `void f(...) => e` in the analyzer.

Re-land, now including adjustment of two tests in pkg/analyzer/test.

R=brianwilkerson@google.com

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

[eernst, 2017-05-11 08:10:23]

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