Compile package:js.{rest, spread} helpers

Compile package:js.{rest, spread} helpers.

Note: need new package:js release before merging this in (https://github.com/dart-lang/sdk/issues/24623). Maybe it's time to think again about sneaking JsName + JsPeerInterface into it? (https://github.com/dart-lang/dev_compiler/issues/135)

BUG=https://github.com/dart-lang/dev_compiler/issues/310
R=jmesserly@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/79e9804af0a12a8cc10fc93c8415668b1619ffa2

[ochafik, 2015-10-29 16:03:52]

Description was changed from

==========
Compile package:js.{rest, spread} helpers

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

to

==========
Compile package:js.{rest, spread} helpers.

Note: need new package:js release before merging this in
(https://github.com/dart-lang/sdk/issues/24623). Maybe it's time to think again
about sneaking JsName + JsPeerInterface into it?
(https://github.com/dart-lang/dev_compiler/issues/135)

BUG=https://github.com/dart-lang/dev_compiler/issues/310
==========

[ochafik, 2015-10-29 16:03:52]

ochafik@google.com changed reviewers:
+ jacobr@google.com, jmesserly@google.com

[ochafik, 2015-10-29 16:07:23]

Hi guys, prepping up for rest+spread, WDYT?

[Jennifer Messerly, 2015-10-29 16:34:41]

LGTM!

I dunno about the version stuff ... maybe Jacob knows?

I'd be totally fine bumping our min SDK version to be some -dev build. We're
pretty lock step with SDK these days (for bleeding edge analyzers).

[Jennifer Messerly, 2015-10-29 16:35:12]

On 2015/10/29 16:34:41, John Messerly wrote:
> LGTM!
> 
> I dunno about the version stuff ... maybe Jacob knows?
> 
> I'd be totally fine bumping our min SDK version to be some -dev build. We're
> pretty lock step with SDK these days (for bleeding edge analyzers).

and actually, I think Travis is testing -dev channel. Maybe push a branch and
see if it likes this version of package:js?

[ochafik, 2015-10-29 17:44:19]

On 2015/10/29 16:35:12, John Messerly wrote:
> On 2015/10/29 16:34:41, John Messerly wrote:
> > LGTM!
> > 
> > I dunno about the version stuff ... maybe Jacob knows?
> > 
> > I'd be totally fine bumping our min SDK version to be some -dev build. We're
> > pretty lock step with SDK these days (for bleeding edge analyzers).
> 
> and actually, I think Travis is testing -dev channel. Maybe push a branch and
> see if it likes this version of package:js?

Thanks John!

Updated to point to shiny new js-0.6.0-beta.6 (thanks Jacob for publishing!).
(& rebased / reformatted)

[Jennifer Messerly, 2015-10-29 17:49:27]

lgtm * 2

[ochafik, 2015-10-29 18:44:15]

Committed patchset #2 (id:20001) manually as
79e9804af0a12a8cc10fc93c8415668b1619ffa2 (presubmit successful).