Closed Bug 1371047 Opened 7 years ago Closed 7 years ago

Move top-level loops into helper functions or IIFEs

Categories

(Toolkit :: Add-ons Manager, enhancement)

Product:
Toolkit
Component:
Add-ons Manager
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: kmag, Unassigned)

References

(Blocks 1 open bug)

Details

According to bug 1167472, it's going to be some time before we have any JIT support for scopes with ES6 class definitions. This means that any top-level loops in modules that use ES6 classes will be fully-interpreted, and execute much slower than they would otherwise.

In cases like XPIProvider.jsm and XPIProviderUtils.js, this is likely to be a big deal, since we have a significant number of top-level loops. We should move these and any other JITtable code (including things like array map/forEach operations) into shared helper functions, IIFEs, or init methods, so that they can bet JITted.
Component: WebExtensions: General → Add-ons Manager
Based on bug 1167472 comment 18, this should no longer be necessary.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.