getOriginalLocation can be expensive

RESOLVED FIXED in Firefox 67

Status

enhancement
RESOLVED FIXED
5 months ago
3 months ago

People

(Reporter: jlast, Unassigned)

Tracking

unspecified
Firefox 67
Dependency tree / graph

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(1 attachment)

there are 80K generated breakpoint positions in debugger.js using the flush/task strategy can use a lot of resources.

doing one invoke for getOriginalLocations is much cheaper.

Just saw this in my (500 line app code) nextjs/typescript app, where it took 5+sec to load the breakpoints (which doesn't have a loading indicator, so is very opaque): https://perfht.ml/2CdhaEu

Pushed by lsmyth@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/985e05a4b540
getOriginalLocation can be expensive. r=loganfsmyth
Pushed by lsmyth@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0d55cac5f346
getOriginalLocation can be expensive. r=loganfsmyth
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Flags: needinfo?(jlaster)
You need to log in before you can comment on or make changes to this bug.