Closed Bug 1005184 Opened 6 years ago Closed 3 years ago
Parent should handle Actor Destroy
No description provided.
Assignee: nobody → jduell.mcbugs
Turns out this is a non-issue. As :billm tells me: "By itself, ActorDestroy isn't too important for sub-protocols that are entirely managed by IPDL (meaning they're deleted inside DeallocPFoo). The destructor will be invoked pretty shortly (i.e., same turn of the event loop) after ActorDestroy is called, so you're not missing much by ignoring ActorDestroy."
Remove "To fix" comments in code and replace with explanation why we have nothing to fix here.
Attachment #8784181 - Flags: review+
This time with the actual patch :)
Please use commit messages that explain what the patch is doing. The one for the attached patch is completely non-informative.
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/4da60cb24006 Clarify comments in Necko ActorDestroy functions clarifying that nothing needs to be implemented. r=jduell
You need to log in before you can comment on or make changes to this bug.