Error when sorting a Proxy with an array as target
Reported by
marco.m...@gmail.com,
Mar 3 2016
|
|||||||||
Issue description
Chrome Version : 49.0.2623.75 (Build ufficiale) (a 64 bit)
Other browsers tested:
Firefox: OK
IE: OK
What steps will reproduce the problem?
1. Execute this JS code:
var pArray = new Proxy([3,2,1], {});
pArray.sort()
What is the expected result?
I would expect that pArray gets sorted
What happens instead?
An error is raised: "Uncaught illegal access"
I have tested other array methods (such as push, pop, splice, shift, unshift, reverse) and everything works fine.
,
Mar 3 2016
,
Mar 4 2016
,
Mar 4 2016
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/86c955fee0aa15c37a7af1bd7dcb0a8bf5a6d92c commit 86c955fee0aa15c37a7af1bd7dcb0a8bf5a6d92c Author: neis <neis@chromium.org> Date: Fri Mar 04 14:42:15 2016 Fix Array.prototype.sort on proxies. BUG= chromium:591699 LOG=n R=rossberg Review URL: https://codereview.chromium.org/1764953002 Cr-Commit-Position: refs/heads/master@{#34498} [modify] https://crrev.com/86c955fee0aa15c37a7af1bd7dcb0a8bf5a6d92c/src/runtime/runtime-array.cc [modify] https://crrev.com/86c955fee0aa15c37a7af1bd7dcb0a8bf5a6d92c/test/mjsunit/array-sort.js
,
Mar 4 2016
,
Mar 9 2016
,
Mar 9 2016
,
Mar 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/c60edba4a1b7eff59825c6e2d2b9415ca57dbd31 commit c60edba4a1b7eff59825c6e2d2b9415ca57dbd31 Author: neis <neis@chromium.org> Date: Wed Mar 09 09:51:37 2016 Version 4.9.385.30 (cherry-pick) Merged 86c955fee0aa15c37a7af1bd7dcb0a8bf5a6d92c Fix Array.prototype.sort on proxies. BUG= chromium:591699 LOG=N R=hablich@chromium.org NOTRY=true NOPRESUBMIT=true Review URL: https://codereview.chromium.org/1775343002 Cr-Commit-Position: refs/branch-heads/4.9@{#36} Cr-Branched-From: 2fea296569597e5064f81fd8fce58f1848de261a-refs/heads/4.9.385@{#1} Cr-Branched-From: 0c1430ac2b65847559d6a09f883ee7e5a91063c9-refs/heads/master@{#33306} [modify] https://crrev.com/c60edba4a1b7eff59825c6e2d2b9415ca57dbd31/include/v8-version.h [modify] https://crrev.com/c60edba4a1b7eff59825c6e2d2b9415ca57dbd31/src/runtime/runtime-array.cc [modify] https://crrev.com/c60edba4a1b7eff59825c6e2d2b9415ca57dbd31/test/mjsunit/array-sort.js
,
Mar 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/b5c05a57ad8525d1cc2dd64d335798a1ba54ff83 commit b5c05a57ad8525d1cc2dd64d335798a1ba54ff83 Author: neis <neis@chromium.org> Date: Thu Mar 10 09:34:42 2016 Version 5.0.71.14 (cherry-pick) Merged 86c955fee0aa15c37a7af1bd7dcb0a8bf5a6d92c Fix Array.prototype.sort on proxies. BUG= chromium:591699 LOG=N R=hablich@chromium.org NOTRY=true NOPRESUBMIT=true Review URL: https://codereview.chromium.org/1777203002 Cr-Commit-Position: refs/branch-heads/5.0@{#20} Cr-Branched-From: ad16e6c2cbd2c6b0f2e8ff944ac245561c682ac2-refs/heads/5.0.71@{#1} Cr-Branched-From: bd9df50d75125ee2ad37b3d92c8f50f0a8b5f030-refs/heads/master@{#34215} [modify] https://crrev.com/b5c05a57ad8525d1cc2dd64d335798a1ba54ff83/include/v8-version.h [modify] https://crrev.com/b5c05a57ad8525d1cc2dd64d335798a1ba54ff83/src/runtime/runtime-array.cc [modify] https://crrev.com/b5c05a57ad8525d1cc2dd64d335798a1ba54ff83/test/mjsunit/array-sort.js
,
Mar 10 2016
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by b...@chromium.org
, Mar 3 2016