Closed Bug 1536196 Opened 5 years ago Closed 5 years ago

loadSourceText should probably be responsible for pretty-printing files

Categories

(DevTools :: Debugger, enhancement)

enhancement
Not set
normal

Tracking

(firefox68 fixed)

RESOLVED FIXED
Firefox 68
Tracking Status
firefox68 --- fixed

People

(Reporter: jlast, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Right now pretty-printing is its own action that kind of hacks its way into the source-loading process. This isn't idea because it means that the promise returned by loadSourceText can't consistently handle the case where a file is in the process of being pretty-printed, since don't have a way to know when pretty-printing completes.

CC https://github.com/firefox-devtools/debugger/issues/8071

Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/416d1c916c85
loadSourceText should probably be responsible for pretty-printing files. r=loganfsmyth
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: