Closed
Bug 1621914
Opened 4 years ago
Closed 4 years ago
Remove custom data rendering in the Response panel
Categories
(DevTools :: Netmonitor, defect, P2)
DevTools
Netmonitor
Tracking
(firefox77 fixed)
RESOLVED
FIXED
Firefox 77
Tracking | Status | |
---|---|---|
firefox77 | --- | fixed |
People
(Reporter: Honza, Assigned: bomsy)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
The response panel is using <pre> element to render responses bigger than devtools.netmonitor.response.ui.limit
I was testing this with patch from bug 1613885 and since Code Mirror is now properly using its virtualization is faster to use it at all times.
We should also verify whether we still need that devtools.netmonitor.response.ui.limit
pref or not.
Honza
Comment 1•4 years ago
|
||
That would be great. I also saw comparable performance with code mirror and long text (with syntax highlighting disabled for long texts, which Debugger does)
Reporter | ||
Comment 2•4 years ago
|
||
I am marking as P2, it's performance improvement
Honza
Priority: P3 → P2
Assignee | ||
Updated•4 years ago
|
Assignee: nobody → hmanilla
Assignee | ||
Comment 3•4 years ago
|
||
Remove PRE for rendering large content, to take advantage of
virtualization in codemirror.
Pushed by hmanilla@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/eca1a3d272be Remove PRE for rendering large content r=Honza,perftest-reviewers,Bebe
Comment 5•4 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox77:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 77
You need to log in
before you can comment on or make changes to this bug.
Description
•