Closed
Bug 1599104
Opened 5 years ago
Closed 5 years ago
More JSFunction cleanups to support LazyScript/JSScript merge
Categories
(Core :: JavaScript Engine, task, P2)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla72
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: tcampbell, Assigned: tcampbell)
References
Details
Attachments
(3 files)
A few more patches to hide the underlying LazyScript/JSScript from users of JSFunction. This makes the transistion to a unified script easier.
Assignee | ||
Comment 1•5 years ago
|
||
The 'function()' method is already defined on BaseScript, so this is
straightforward to fix.
Assignee | ||
Comment 2•5 years ago
|
||
Hide the check for LazyScript vs JSScript inside an accessor function.
Depends on D54525
Assignee | ||
Comment 3•5 years ago
|
||
Split out the self-hosted handling from delazifyLazilyInterpretedFunction
since it will need to be handled differently when LazyScript merges with
JSScript.
Depends on D54526
Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ae08d265de84
Simplify JSFunction::maybeCanonicalFunction(). r=jandem
https://hg.mozilla.org/integration/autoland/rev/d4d888235266
Add JSFunction::enclosingScope(). r=jandem
https://hg.mozilla.org/integration/autoland/rev/da943611acea
Add JSFunction::delazifySelfHostedLazyFunction(). r=jandem
Comment 5•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ae08d265de84
https://hg.mozilla.org/mozilla-central/rev/d4d888235266
https://hg.mozilla.org/mozilla-central/rev/da943611acea
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox72:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in
before you can comment on or make changes to this bug.
Description
•