Closed Bug 1772347 Opened 4 years ago Closed 4 years ago

Content process targets destruction isn't notified to the client

Categories

(DevTools :: Framework, defect, P3)

defect

Tracking

(firefox104 fixed)

RESOLVED FIXED
104 Branch
Tracking Status
firefox104 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

This only happens when we destroy the target, but the content process itself isn't destroyed.

When the process is destroyed, the following code will kicks in and notify about the target destruction:
https://searchfox.org/mozilla-central/rev/c8e15e17bc6fd28f558c395c948a6251b38774ff/devtools/server/actors/watcher/target-helpers/process-helper.js#80-86

In bug 1772310, we will start destroying content process target actors, while the process keeps running and start having some side effects because of this.

That, when the target is destroyed for any reason other than the process destruction.
For example, when we stop observing for content process targets.

Severity: -- → S3
Priority: -- → P3
Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a6d7efb82917 [devtools] Ensure that content process targets destruction is notified to the client. r=jdescottes

Backed out changeset a6d7efb82917 (Bug 1772347) for causing devtools failres.
Backout link
Push with failures <--> dt8
Failure Log

Flags: needinfo?(poirot.alex)
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7241fe6d58cf [devtools] Ensure that content process targets destruction is notified to the client. r=jdescottes
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 104 Branch
Flags: needinfo?(poirot.alex)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: