Open
Bug 1441785
Opened 7 years ago
Updated 2 years ago
Improve performance of VisibilityHelper
Categories
(DevTools :: Framework, enhancement, P2)
DevTools
Framework
Tracking
(Not tracked)
NEW
People
(Reporter: Honza, Unassigned)
References
(Blocks 1 open bug)
Details
This is a follow up for bug 1419350
Visibility helper is nicely improving the performance by avoiding useless re-rendering if a panel is running in background.
But, React is still spending significant time when state changes. It would be great if we could disconnect components from Redux store temporarily and safe even more CPU cycles.
---
From this comment:
https://bugzilla.mozilla.org/show_bug.cgi?id=1419350#c26
The vast majority of react.js is delegated to react-dom.js:enqueueSetState
By looking at the stacks, it looks like it is only to update the VisibilityHandler component.
We can see its shouldComponentUpdate method being called.
Then comes reselect and all the filter function, we spend 34ms in them.
Honza
Updated•7 years ago
|
Updated•7 years ago
|
Product: Firefox → DevTools
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•