1600663 - ContentParent::MarkAsDead should alert Android to unbind content process service

Status: RESOLVED FIXED

(GeckoView Graveyard :: Sandboxing, defect, P1)

Description

[(No longer employed by Mozilla) Aaron Klotz]

ContentParent::MarkAsDead is used by Gecko's content process management to remove a defunct ContentParent object from the list of running content processes.

Note that the actual content process that is associated with this object might still be running, albeit it should be shutting down. The point is that, as far as Gecko is concerned, the process is no a longer viable candidate for running content. From GeckoView's standpoint, we should also be removing that process from candidate content processes by unbinding its service.

(This might also explain the need for the dom.ipc.keepProcessesAlive.web pref that Snorp was discussing with me last week. If we fix this, we can try getting rid of that.)

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1600663: Hook GV process management into ContentParent; r=#geckoview-reviewers

We need to propagate ContentParent::MarkAsDead up to the Android java layer
so that it has a view of the state of content processes that is consistent
with the view of Gecko's content process management.

Depends on D57840

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bf89cb3c0142
Hook GV process management into ContentParent; r=geckoview-reviewers,snorp

Comment 3

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/bf89cb3c0142

Comment 4

[Chris Peterson [:cpeterson]]

Moving content process management bugs to the new GeckoView::Sandboxing component.