Closed
Bug 1567633
Opened 6 years ago
Closed 6 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$1is high, so switching some classes to pure components or usingshouldComponentUpdatemight reduce overhead. setStatetakes the biggest chunk of time, so debouncingonFrameReceived(maybe usingrequestIdleCallback) might also free some main thread and bundle table updates.
| Assignee | ||
Comment 1•6 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•6 years ago
|
Assignee: nobody → E0032242
Status: NEW → ASSIGNED
Comment 3•6 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Updated•6 years ago
|
QA Whiteboard: [qa-70b-p2]
You need to log in
before you can comment on or make changes to this bug.
Description
•