Closed Bug 1943339 Opened 1 month ago Closed 1 month ago

Improve react redux selectors in netmonitor to unnecessary renders

Categories

(DevTools :: Netmonitor, task, P3)

task

Tracking

(firefox136 fixed)

RESOLVED FIXED
136 Branch
Tracking Status
firefox136 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

Details

Attachments

(4 files)

Extracting some patches from 1849920 to land them earlier

The individual input selectors of reselect selectors should be as close as possible to the
data used by the selector, so that we can efficiently memoize the selector.
Since this selector returns an object which will always fail shallow comparisons it's important to
make sure we memoize efficiently.

Depends on D234517

Update input selectors to memoize selector functions more efficiently.

Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b3f1de9a3ea3 [devtools] Fix memoization issues for netmonitor getColumns selector r=devtools-reviewers,ochameau https://hg.mozilla.org/integration/autoland/rev/deba5965ab0d [devtools] Update netmonitor requests selectors to improve memoization r=devtools-reviewers,ochameau https://hg.mozilla.org/integration/autoland/rev/6f6c065d3c1d [devtools] Improve request selectors to avoid rebuilding filters for each request r=ochameau,devtools-reviewers https://hg.mozilla.org/integration/autoland/rev/cb8f5ec29868 [devtools] Avoid unnecessary renders due to blockedUrls prop r=devtools-reviewers,bomsy
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: