Closed Bug 1225477 Opened 9 years ago Closed 9 years ago

Support watching new process creation

Categories

(DevTools :: Framework, defect)

Product:
DevTools
Component:
Framework
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox45 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 45
Tracking Flags:
Tracking Status
firefox45 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(1 file, 1 obsolete file)

patch v2
(deleted), patch
ochameau
: review+
Details | Diff | Splinter Review
The root actor allows to list processes and create a ChildProcessActor for each of them. But it doesn't send event to the client whenever a new child is created. That prevent tracking new processes.
Assignee: nobody → poirot.alex
Attached patch patch v1 (obsolete) (deleted) — — Splinter Review
Up for a quick review before leaving ? ;) Just tell me, otherwise I will seek for someone else.
Attachment #8691958 - Flags: review?(jryans)
Comment on attachment 8691958 [details] [diff] [review] patch v1 Review of attachment 8691958 [details] [diff] [review]: ----------------------------------------------------------------- It seems correct, but I looked quickly... Not too worried since it seems like the RDP API surface does not change, so we can tweak later if needed.
Attachment #8691958 - Flags: review?(jryans) → review+
Attached patch patch v2 (deleted) — — Splinter Review
Rebased.
Attachment #8694848 - Flags: review+
Attachment #8691958 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/3858001d34b3
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: