Closed
Bug 1567633
Opened 5 years ago
Closed 5 years ago
High-frequency messages cause multi-second hangs
Categories
(DevTools :: Netmonitor, defect, P3)
DevTools
Netmonitor
Tracking
(firefox70 fixed)
RESOLVED
FIXED
Firefox 70
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: Harald, Assigned: tanhengyeow)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
STR:
- Open https://puzzling-rake.glitch.me/ (http://tankanarchy.herokuapp.com/ for a game that sends even more)
- Observe WebSocket messages
AR:
The UI freezes for multiple seconds while messages are rendered in chunks.
Bug 1561631 might help with this somewhat, but only if the bottleneck is the amount of elements; which might not be the case.
Profile: https://perfht.ml/2YgfAh8 (tankanarchy: https://perfht.ml/2Ybjnwd)
- React's
diffProperties$1
is high, so switching some classes to pure components or usingshouldComponentUpdate
might reduce overhead. setState
takes the biggest chunk of time, so debouncingonFrameReceived
(maybe usingrequestIdleCallback
) might also free some main thread and bundle table updates.
Assignee | ||
Comment 1•5 years ago
|
||
Triggers addFrames in batches
Pushed by jodvarko@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/225f4a11bbb1 High-frequency messages cause multi-second hangs. r=Honza
Updated•5 years ago
|
Assignee: nobody → E0032242
Status: NEW → ASSIGNED
Comment 3•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Updated•5 years ago
|
QA Whiteboard: [qa-70b-p2]
You need to log in
before you can comment on or make changes to this bug.
Description
•