Remove the GetLengthProperty overload that returns uint32_t
Categories
(Core :: JavaScript Engine, task, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox89 | --- | fixed |
People
(Reporter: jandem, Assigned: jandem)
References
Details
Attachments
(9 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review |
Lars noticed this causing problems for some tests.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
These clamp to or check for ARGS_LENGTH_MAX so there's no change in behavior.
Depends on D111381
Assignee | ||
Comment 3•4 years ago
|
||
These functions aren't available to content and we don't want to support
large lengths there, so just report an exception.
Depends on D111382
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D111383
Assignee | ||
Comment 5•4 years ago
|
||
Depends on D111384
Assignee | ||
Comment 6•4 years ago
|
||
Depends on D111385
Assignee | ||
Comment 7•4 years ago
|
||
This is always an ArrayObject, so get its length() directly.
Depends on D111386
Assignee | ||
Comment 8•4 years ago
|
||
Depends on D111387
Assignee | ||
Comment 9•4 years ago
|
||
There are a number of callers in Gecko, most of them are using to iterate over an object
expected to be an array. This should be okay for now; if there are callers that need the
full length we can add a separate API that returns uint64_t.
This lets us remove ToLengthClamped and the GetLengthProperty overload returning uint32_t.
Depends on D111388
Comment 10•4 years ago
|
||
Pushed by jdemooij@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/93d64ac021f9 part 1 - Make GetLengthProperty overload returning uint64_t available outside Array.cpp. r=lth https://hg.mozilla.org/integration/autoland/rev/3458449ccb39 part 2 - Use GetLengthProperty overload returning uint64_t for function calls. r=lth https://hg.mozilla.org/integration/autoland/rev/ad3456e4f2bc part 3 - Use GetLengthProperty overload returning uint64_t for some internal functions. r=lth https://hg.mozilla.org/integration/autoland/rev/9d20ba60f0ee part 4 - Use GetLengthProperty overload returning uint64_t for TypedArray fromObject. r=lth https://hg.mozilla.org/integration/autoland/rev/57579fea8a65 part 5 - Use GetLengthProperty overload returning uint64_t for TypedArray set method. r=lth https://hg.mozilla.org/integration/autoland/rev/bb0fc8f4bbac part 6 - Use GetLengthProperty overload returning uint64_t for CreateFilteredListFromArrayLike. r=lth https://hg.mozilla.org/integration/autoland/rev/91f155b2913f part 7 - Use ArrayObject::length instead of GetLengthProperty for module namespace exports object. r=lth https://hg.mozilla.org/integration/autoland/rev/6da24a2d6334 part 8 - Add GetLengthPropertyForArray to the JSON code. r=lth https://hg.mozilla.org/integration/autoland/rev/79f3f364ed33 part 9 - Report an exception for large lengths from JS::GetArrayLength API. r=lth
Comment 11•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/93d64ac021f9
https://hg.mozilla.org/mozilla-central/rev/3458449ccb39
https://hg.mozilla.org/mozilla-central/rev/ad3456e4f2bc
https://hg.mozilla.org/mozilla-central/rev/9d20ba60f0ee
https://hg.mozilla.org/mozilla-central/rev/57579fea8a65
https://hg.mozilla.org/mozilla-central/rev/bb0fc8f4bbac
https://hg.mozilla.org/mozilla-central/rev/91f155b2913f
https://hg.mozilla.org/mozilla-central/rev/6da24a2d6334
https://hg.mozilla.org/mozilla-central/rev/79f3f364ed33
Description
•