getOriginalLocation can be expensive
Categories
(DevTools :: Debugger, enhancement)
Tracking
(firefox67 fixed)
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: jlast, Unassigned)
References
Details
Attachments
(1 file)
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.
Reporter | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
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
Comment 4•6 years ago
|
||
Backed out changeset 985e05a4b540 (bug 1533813) for Debugger failure. CLOSED TREE
Log:
https://treeherder.mozilla.org/logviewer.html#?job_id=233253485&repo=autoland
Push with failure:
https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=233218358&revision=985e05a4b54024ae40be6d94aaf8d9fb3d66f9b2
Backout:
https://hg.mozilla.org/integration/autoland/rev/02b387ef172ee8e00ba33eb630def6cb0784b05b
Comment 6•6 years ago
|
||
bugherder |
Reporter | ||
Updated•6 years ago
|
Description
•