Closed Bug 1580231 Opened 5 years ago Closed 5 years ago

Avoid creating a new process script everytime we fetch process targets

Categories

(DevTools :: Framework, enhancement, P2)

enhancement

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1582577

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

Priority: -- → P2
Whiteboard: dt-fission

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.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
Whiteboard: dt-fission
You need to log in before you can comment on or make changes to this bug.