Avoid creating a new process script everytime we fetch process targets
Categories
(DevTools :: Framework, enhancement, P2)
Tracking
(Not tracked)
People
(Reporter: ochameau, Unassigned)
References
(Blocks 1 open bug)
Details
This code:
https://searchfox.org/mozilla-central/rev/250f5cc9fb8bdcbb6b23d2a06acfd48addb2f99b/devtools/server/actors/process.js#66
currently creates one process script each time we call RootFront.listProcesses,
so, each time a new process is created/destroyed.
We should avoid doing that as it pollutes the source tree of the omniscient toolbox.
We should be able to listen for these two events instead:
https://searchfox.org/mozilla-central/rev/250f5cc9fb8bdcbb6b23d2a06acfd48addb2f99b/dom/ipc/ContentParent.cpp#1313-1314
https://searchfox.org/mozilla-central/rev/250f5cc9fb8bdcbb6b23d2a06acfd48addb2f99b/dom/ipc/ContentParent.cpp#1685-1686
Updated•5 years ago
|
Reporter | ||
Comment 1•5 years ago
|
||
This has been fixed in bug 1582577.
We aren't using the two mentioned events from ContentParent, but at least, we no longer create a new process script each time listProcesses is called.
Updated•5 years ago
|
Description
•