Fix the receiver check in the HandleFastApiCall builtin.

Fix the receiver check in the HandleFastApiCall builtin.

CompatibleReceiverCheck used by the HandleFastApiCall builtin was terminating with failure upon encountering a hidden prototype.

It should actually stop iterating on the first non-hidden prototype.

BUG=

Committed: https://crrev.com/71129d5b4323902ab9702ae3e5e123b3c59d5dfd
Cr-Commit-Position: refs/heads/master@{#33294}

[epertoso, 2016-01-12 16:43:51]

Description was changed from

==========
CompatibleReceiverCheck for the HandleFastApiCall builtin was terminating with
failure upon encountering a hidden prototype.

It should actually stop iterating on the first non-hidden prototype.

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

to

==========
CompatibleReceiverCheck used by the HandleFastApiCall builtin was terminating
with failure upon encountering a hidden prototype.

It should actually stop iterating on the first non-hidden prototype.

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

[epertoso, 2016-01-12 16:43:52]

epertoso@chromium.org changed reviewers:
+ bmeurer@chromium.org, jochen@chromium.org

[epertoso, 2016-01-12 16:43:52]

epertoso@chromium.org changed required reviewers:
+ bmeurer@chromium.org

[epertoso, 2016-01-12 16:44:01]

The CQ bit was checked by epertoso@chromium.org

[commit-bot: I haz the power, 2016-01-12 16:44:13]

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

[commit-bot: I haz the power, 2016-01-12 16:44:16]

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

[commit-bot: I haz the power, 2016-01-12 16:44:17]

All required reviewers (with asterisk prefixes) have not yet approved this CL.

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[epertoso, 2016-01-12 16:46:08]

The CQ bit was checked by epertoso@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-12 16:46:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1576423003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1576423003/1

[commit-bot: I haz the power, 2016-01-12 17:53:58]

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

[commit-bot: I haz the power, 2016-01-12 17:53:59]

Dry run: Try jobs failed on following builders:
  v8_linux_arm_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel/builds/12289)

[Benedikt Meurer, 2016-01-13 05:34:16]

lgtm

[epertoso, 2016-01-13 09:11:34]

The CQ bit was checked by epertoso@chromium.org

[commit-bot: I haz the power, 2016-01-13 09:11:41]

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

[commit-bot: I haz the power, 2016-01-13 09:48:21]

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

[commit-bot: I haz the power, 2016-01-13 09:48:22]

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

[epertoso, 2016-01-13 10:07:08]

The CQ bit was checked by epertoso@chromium.org

[commit-bot: I haz the power, 2016-01-13 10:07:12]

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

[Michael Achenbach, 2016-01-13 10:39:02]

Please fix the CL description. First line becomes the commit message and should
be equal to rietveld's title.

PS. I think the v8_linux_rel test failure is real.

[commit-bot: I haz the power, 2016-01-13 10:42:55]

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

[commit-bot: I haz the power, 2016-01-13 10:42:55]

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

[epertoso, 2016-01-13 11:10:01]

Description was changed from

==========
CompatibleReceiverCheck used by the HandleFastApiCall builtin was terminating
with failure upon encountering a hidden prototype.

It should actually stop iterating on the first non-hidden prototype.

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

to

==========
Fix the receiver check in the HandleFastApiCall builtin.

CompatibleReceiverCheck used by the HandleFastApiCall builtin was terminating
with failure upon encountering a hidden prototype.

It should actually stop iterating on the first non-hidden prototype.

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

[epertoso, 2016-01-13 11:10:37]

On 2016/01/13 at 10:39:02, machenbach wrote:
> Please fix the CL description. First line becomes the commit message and
should be equal to rietveld's title.
> 
> PS. I think the v8_linux_rel test failure is real.

Yeah, it's related to the code aging prologue. I'm looking into it.

[epertoso, 2016-01-14 11:50:33]

The CQ bit was checked by epertoso@chromium.org

[epertoso, 2016-01-14 11:50:34]

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

[commit-bot: I haz the power, 2016-01-14 11:50:38]

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

[commit-bot: I haz the power, 2016-01-14 12:23:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-14 12:24:11]

Description was changed from

==========
Fix the receiver check in the HandleFastApiCall builtin.

CompatibleReceiverCheck used by the HandleFastApiCall builtin was terminating
with failure upon encountering a hidden prototype.

It should actually stop iterating on the first non-hidden prototype.

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

to

==========
Fix the receiver check in the HandleFastApiCall builtin.

CompatibleReceiverCheck used by the HandleFastApiCall builtin was terminating
with failure upon encountering a hidden prototype.

It should actually stop iterating on the first non-hidden prototype.

BUG=

Committed: https://crrev.com/71129d5b4323902ab9702ae3e5e123b3c59d5dfd
Cr-Commit-Position: refs/heads/master@{#33294}
==========

[commit-bot: I haz the power, 2016-01-14 12:24:12]

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