Convert primitive receivers for API property callbacks

Convert primitive receivers for API property callbacks

They're always in sloppy mode, so always do the conversion

BUG=chromium:609134
R=bmeurer@chromium.org,verwaest@chromium.org
LOG=n

Committed: https://crrev.com/127d6781d91400397e15ba7302483643fb26c1da
Cr-Commit-Position: refs/heads/master@{#36084}

[Benedikt Meurer, 2016-05-06 12:15:58]

LGTM.

[jochen (gone - plz use gerrit), 2016-05-06 12:19:02]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-05-06 12:19:07]

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

[commit-bot: I haz the power, 2016-05-06 12:39:55]

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

[commit-bot: I haz the power, 2016-05-06 12:39:56]

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

[jochen (gone - plz use gerrit), 2016-05-06 13:40:05]

The CQ bit was checked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-05-06 13:40:05]

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/1960663002/#ps40001
(title: "updates")

[commit-bot: I haz the power, 2016-05-06 13:40:15]

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

[jochen (gone - plz use gerrit), 2016-05-06 13:56:39]

https://codereview.chromium.org/1960663002/diff/40001/src/accessors.cc
File src/accessors.cc (right):

https://codereview.chromium.org/1960663002/diff/40001/src/accessors.cc#newcod...
src/accessors.cc:264: // We have a slight impedance mismatch between the
external API and the way we
slight? in the sense of how sloppy and strict are slightly different? :-/

[Toon Verwaest, 2016-05-06 14:09:40]

Not all sloppy functions convert receivers. Builtins are defined to not do it,
for example. Are you sure we want this?

[commit-bot: I haz the power, 2016-05-06 14:10:18]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-06 14:11:33]

Description was changed from

==========
Convert primitive receivers for API property callbacks

They're always in sloppy mode, so always do the conversion

BUG=chromium:609134
R=bmeurer@chromium.org,verwaest@chromium.org
LOG=n
==========

to

==========
Convert primitive receivers for API property callbacks

They're always in sloppy mode, so always do the conversion

BUG=chromium:609134
R=bmeurer@chromium.org,verwaest@chromium.org
LOG=n

Committed: https://crrev.com/127d6781d91400397e15ba7302483643fb26c1da
Cr-Commit-Position: refs/heads/master@{#36084}
==========

[commit-bot: I haz the power, 2016-05-06 14:11:34]

Patchset 3 (id:??) landed as
https://crrev.com/127d6781d91400397e15ba7302483643fb26c1da
Cr-Commit-Position: refs/heads/master@{#36084}

[jochen (gone - plz use gerrit), 2016-05-06 14:16:51]

On 2016/05/06 at 14:09:40, verwaest wrote:
> Not all sloppy functions convert receivers. Builtins are defined to not do it,
for example. Are you sure we want this?

Well, This is defined as Local<Object> so it better be a JSReceiver...