Closed Bug 1898677 Opened 1 year ago Closed 1 year ago

Do not call ShouldCacheBytecode multiple times

Categories

(Core :: JavaScript Engine, task)

task

Tracking

()

RESOLVED FIXED
128 Branch
Tracking Status
firefox128 --- fixed

People

(Reporter: arai, Assigned: arai)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

mozilla::dom::ScriptLoader::ShouldCacheBytecode is called multiple times from different places for different or overlapping purposes.
https://searchfox.org/mozilla-central/search?q=ShouldCacheBytecode&path=

This is redundant and also it can cause inconsistency when the condition becomes different (e.g. pref is changed)

Pushed by arai_a@mac.com: https://hg.mozilla.org/integration/autoland/rev/89b121acbe71 Cache the result of bytecode encoding condition. r=nbp
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: