Ship arrow functions

Ship arrow functions

R=rossberg@chromium.org
LOG=Y
BUG=v8:2700
Committed: https://crrev.com/9594cc72eec3ad64506c531106dba6c8fc42bae9
Cr-Commit-Position: refs/heads/master@{#29167}

[wingo, 2015-06-19 12:54:10]

Previously landed in https://codereview.chromium.org/1187173004/, reverted
because a layout test did "return();" to trigger a devtools behavior, but then
matched verbatim on the error, and (); caused a different error after arrow
functions were enabled.  This is fixed to a degree in
https://codereview.chromium.org/1191303002/, but the error still isn't the same
so probably that one failing test will need a new baseline.

[rossberg, 2015-06-19 13:03:50]

lgtm

[rossberg, 2015-06-19 13:06:39]

On 2015/06/19 12:54:10, wingo wrote:
> Previously landed in https://codereview.chromium.org/1187173004/, reverted
> because a layout test did "return();" to trigger a devtools behavior, but then
> matched verbatim on the error, and (); caused a different error after arrow
> functions were enabled.  This is fixed to a degree in
> https://codereview.chromium.org/1191303002/, but the error still isn't the
same
> so probably that one failing test will need a new baseline.

Can you do this rebaseline, or should we help with that?

[wingo, 2015-06-19 13:11:35]

On 2015/06/19 13:06:39, rossberg wrote:
> On 2015/06/19 12:54:10, wingo wrote:
> > Previously landed in https://codereview.chromium.org/1187173004/, reverted
> > because a layout test did "return();" to trigger a devtools behavior, but
then
> > matched verbatim on the error, and (); caused a different error after arrow
> > functions were enabled.  This is fixed to a degree in
> > https://codereview.chromium.org/1191303002/, but the error still isn't the
> same
> > so probably that one failing test will need a new baseline.
> 
> Can you do this rebaseline, or should we help with that?

I'm going to submit a NeedsManualRebaseline patch to blink after
https://codereview.chromium.org/1191303002/ lands linking to crbug.com/502243
(as it seems you need a crbug link).  I'll let you know if I have problems,
thanks.  Just waiting on the other patch now.

[wingo, 2015-06-19 19:02:25]

The CQ bit was checked by wingo@igalia.com

[commit-bot: I haz the power, 2015-06-19 19:02:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1194873002/1

[commit-bot: I haz the power, 2015-06-19 19:55:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-06-19 19:55:26]

Patchset 1 (id:??) landed as
https://crrev.com/9594cc72eec3ad64506c531106dba6c8fc42bae9
Cr-Commit-Position: refs/heads/master@{#29167}