Open Bug 1441785 Opened 6 years ago Updated 1 year ago

Improve performance of VisibilityHelper


(DevTools :: Framework, enhancement, P2)



(Not tracked)


(Reporter: Honza, Unassigned)


(Blocks 1 open bug)


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:

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.

Severity: normal → enhancement
Priority: -- → P2
Product: Firefox → DevTools
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.