Closed Bug 1621914 Opened 4 years ago Closed 4 years ago

Remove custom data rendering in the Response panel

Categories

(DevTools :: Netmonitor, defect, P2)

defect

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

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)

I am marking as P2, it's performance improvement
Honza

Priority: P3 → P2
Blocks: 1622509
No longer blocks: 1622509
Assignee: nobody → hmanilla

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
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 77
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: