Closed Bug 1819350 Opened 2 years ago Closed 2 years ago

prettyFast sanitize could be faster

Categories

(DevTools :: Debugger, task)

task

Tracking

(firefox112 fixed)

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: nchevobbe, Assigned: nchevobbe)

Details

Attachments

(1 file)

https://searchfox.org/mozilla-central/rev/aa3ccd258b64abfd4c5ce56c1f512bc7f65b844c/devtools/client/debugger/src/workers/pretty-print/pretty-fast.js#544-577

we could avoid having the inner function, declare the regex as const, and not create the replacer function for each call

Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8cbaead8a604 [devtools] Simplify prettyFast sanitize function. r=devtools-reviewers,jdescottes
Flags: needinfo?(nchevobbe)
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0a4d2b462bea [devtools] Simplify prettyFast sanitize function. r=devtools-reviewers,jdescottes
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: