[proxies] Adapt and reenable remaining tests in proxies.js

[proxies] Adapt and reenable remaining tests in proxies.js

R=cbruni@chromium.org
BUG=

Committed: https://crrev.com/d006f617d4cbfbcdc545411bbb4127fa53fc47bc
Cr-Commit-Position: refs/heads/master@{#33180}

[Camillo Bruni, 2015-12-21 12:50:05]

lgtm with nit

https://codereview.chromium.org/1543553002/diff/1/test/mjsunit/harmony/proxie...
File test/mjsunit/harmony/proxies.js (right):

https://codereview.chromium.org/1543553002/diff/1/test/mjsunit/harmony/proxie...
test/mjsunit/harmony/proxies.js:1372: assertThrows(() =>
Object.prototype.toString.call(p))
Could you check explicitly for the thrown error?

[neis, 2016-01-07 14:57:48]

The CQ bit was checked by neis@chromium.org

[neis, 2016-01-07 14:57:48]

The patchset sent to the CQ was uploaded after l-g-t-m from cbruni@chromium.org
Link to the patchset: https://codereview.chromium.org/1543553002/#ps20001
(title: "Address comment")

[commit-bot: I haz the power, 2016-01-07 14:57:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1543553002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1543553002/20001

[commit-bot: I haz the power, 2016-01-07 15:00:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-07 15:00:07]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/9522)

[neis, 2016-01-07 15:07:18]

Description was changed from

==========
[proxies] Adapt and reenable remaining tests in proxies.js

R=cbruni@chromium.org
BUG=
==========

to

==========
[proxies] Adapt and reenable remaining tests in proxies.js

R=cbruni@chromium.org
BUG=
==========

[neis, 2016-01-07 15:07:18]

neis@chromium.org changed reviewers:
+ rossberg@chromium.org

[rossberg, 2016-01-08 14:54:43]

lgtm

[neis, 2016-01-08 14:54:59]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-01-08 14:55:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1543553002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1543553002/20001

[commit-bot: I haz the power, 2016-01-08 14:56:39]

Description was changed from

==========
[proxies] Adapt and reenable remaining tests in proxies.js

R=cbruni@chromium.org
BUG=
==========

to

==========
[proxies] Adapt and reenable remaining tests in proxies.js

R=cbruni@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2016-01-08 14:56:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-08 14:57:08]

Description was changed from

==========
[proxies] Adapt and reenable remaining tests in proxies.js

R=cbruni@chromium.org
BUG=
==========

to

==========
[proxies] Adapt and reenable remaining tests in proxies.js

R=cbruni@chromium.org
BUG=

Committed: https://crrev.com/d006f617d4cbfbcdc545411bbb4127fa53fc47bc
Cr-Commit-Position: refs/heads/master@{#33180}
==========

[commit-bot: I haz the power, 2016-01-08 14:57:08]

Patchset 2 (id:??) landed as
https://crrev.com/d006f617d4cbfbcdc545411bbb4127fa53fc47bc
Cr-Commit-Position: refs/heads/master@{#33180}