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)
DevTools
Netmonitor
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
Assignee | ||
Comment 1•1 month ago
|
||
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.
Assignee | ||
Comment 2•1 month ago
|
||
Depends on D234517
Update input selectors to memoize selector functions more efficiently.
Assignee | ||
Comment 3•1 month ago
|
||
Depends on D234519
Assignee | ||
Comment 4•1 month ago
|
||
Depends on D234704
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
Comment 6•1 month ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b3f1de9a3ea3
https://hg.mozilla.org/mozilla-central/rev/deba5965ab0d
https://hg.mozilla.org/mozilla-central/rev/6f6c065d3c1d
https://hg.mozilla.org/mozilla-central/rev/cb8f5ec29868
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
status-firefox136:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•