1631080 - Canonical location for top-level IPC actors to notify GeckoView of actor destruction

Status: NEW

(Core :: IPC, task)

Description

[(No longer employed by Mozilla) Aaron Klotz]

Currently for each type of content process that we use in GeckoView, we need to add code to the top-level parent actor's ActorDestroy method to call into GeckoView's GeckoProcessManager::MarkAsDead to notify GeckoView's Java layer that the process is terminating.

It would be kinda nice if we could just move this code to a single location (say, ITopLevelProtocol) so that we get that notification for free any time we add a new content process type to GeckoView's repertoire.

I think the only thing that really blocks this ATM is that GeckoProcessManager::MarkAsDead requires a GeckoProcessManager::Selector, which requires both the GeckoProcessType and the pid. We'd need a way to retrieve the process type from the protocol implementation.

I suppose I could remove the Selector as an argument, but it provides useful hints that allow us to more efficiently look up the process in question. I'll think about that some more.

This isn't urgent, but would be a nice future-proofing thing to have.