Closed Bug 1566450 Opened 5 years ago Closed 5 years ago

Add descriptor metaclass for the process target type

Categories

(DevTools :: Framework, task, P1)

task

Tracking

(firefox70 fixed)

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: yulia, Assigned: yulia)

References

(Blocks 1 open bug)

Details

(Whiteboard: dt-fission-m1)

Attachments

(3 files)

process, we need to open a connection to it. If we create a list of targets without care, we could suffer a performance hit, since every target requires that we spin up a server in its dedicated process, along with a loader, and all of its associated actors -- when all we want is potentially the URL -- or to just know that it exists. Spinning up all targets at once is almost never necessary, usually we want a list of metadata from the targets so we can iterate over them (for example the about:debugging panel does this). We need an api that allows us to work with listing and selecting targets safely.

This is a first pass at this idea, which will implement a metaclass for processes, which is the only target type that is not represented by a proxy, but by a json object.

WIP for metaclass concept

The best place to start is the test, it outlines what the API looks like.

Pushed by ystartsev@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ea5411c307fb Implement a metaclass for content processes r=ochameau,jdescottes https://hg.mozilla.org/integration/autoland/rev/b0eee59fb22c replace getList process json with processDescriptor; r=ochameau

I am looking into this but it is trickier than I thought. It looks to be due to the destruction order of targets, specifically -- the process target doesn't unmanage itself when destroyed. Not sure how that is happening.

Will keep looking

Flags: needinfo?(ystartsev)
Pushed by ystartsev@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e603094e855a Implement a metaclass for content processes r=ochameau,jdescottes https://hg.mozilla.org/integration/autoland/rev/4c1d22f89c09 replace getList process json with processDescriptor; r=ochameau https://hg.mozilla.org/integration/autoland/rev/8edf73d4926a disable aboutdebugging_process_main_local test on debug builds; r=jdescottes
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Priority: P2 → P1
Whiteboard: dt-fission → dt-fission dt-fission-m1
Whiteboard: dt-fission dt-fission-m1 → dt-fission-m1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: