dart2js: rewrite_async: Don't call JavaScript factory constructor with 'new'

dart2js: rewrite_async: Don't call JavaScript factory constructor with 'new'

'new' does an unused allocation in this case, since the factory is a static method.

R=efortuna@google.com

Also: use literal syntax for lists.
Committed: https://github.com/dart-lang/sdk/commit/7f3b385d4ce6c09861791cdd02bad904c9f2164a

[sra1, 2017-04-29 17:02:48]

Description was changed from

==========
dart2js: rewrite_async: Daon't call JavaScript factory constructor with 'new'

'new' does an unused allocation in this case.

Also: use literal syntax for lists.
==========

to

==========
dart2js: rewrite_async: Don't call JavaScript factory constructor with 'new'

'new' does an unused allocation in this case, since the factory is a static
method.

Also: use literal syntax for lists.
==========

[sra1, 2017-04-29 17:02:49]

sra@google.com changed reviewers:
+ sigurdm@google.com

[sra1, 2017-05-01 19:47:53]

sra@google.com changed reviewers:
+ sigmund@google.com
- sigurdm@google.com

[Emily Fortuna, 2017-05-01 22:48:54]

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

[Emily Fortuna, 2017-05-01 22:48:54]

lgtm

[sra1, 2017-05-01 23:51:14]

Description was changed from

==========
dart2js: rewrite_async: Don't call JavaScript factory constructor with 'new'

'new' does an unused allocation in this case, since the factory is a static
method.

Also: use literal syntax for lists.
==========

to

==========
dart2js: rewrite_async: Don't call JavaScript factory constructor with 'new'

'new' does an unused allocation in this case, since the factory is a static
method.

R=efortuna@google.com

Also: use literal syntax for lists.
Committed:
https://github.com/dart-lang/sdk/commit/7f3b385d4ce6c09861791cdd02bad904c9f2164a
==========

[sra1, 2017-05-01 23:51:15]

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

[Siggi Cherem (dart-lang), 2017-05-02 19:22:05]

lgtm