Closed Bug 1591405 Opened 5 years ago Closed 5 years ago

Add BaseScript::function()

Categories

(Core :: JavaScript Engine, task, P2)

task

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: tcampbell, Assigned: tcampbell)

References

Details

Attachments

(1 file)

Split off from Bug 1583816 since this was a bit more involved and the first pieces already landed.

Instead of checking for canonical function using scope chain, we can
directly use the functionOrGlobal field. By moving this to the
BaseScript class we can also simplify the debugger acccess.

Attachment #9104208 - Attachment description: Bug 1591405 - Add BaseScript::functionNonDelazifying() and simplify code → Bug 1591405 - Add BaseScript::function() and simplify code
Depends on: 1593354
Summary: Add BaseScript::functionNonDelazifying() → Add BaseScript::function()
Attachment #9104208 - Attachment description: Bug 1591405 - Add BaseScript::function() and simplify code → Bug 1591405 - Add BaseScript::function() and simplify code. r?jandem
Pushed by tcampbell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8d0f2ca33ee8 Add BaseScript::function() and simplify code. r=jandem
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: