1562761 - Bind the Android process launcher to a single owner thread

Status: RESOLVED FIXED

(Core :: IPC, task, P2)

Description

[Bobby Holley (:bholley)]

Our current setup for process launching on Android involves the IPC machinery dispatching via a thread pool and invoking a synchronous process launch method in Java. That Java machinery then uses explicit synchronization to avoid data races.

I don't think this setup is going to scale well to lots of concurrent process launches and service reuse. The Right Way to do this is to have an owner thread, and replace the current blocking bits with asynchronous callbacks. I'm working on this during my free moments.

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1562761 - Add support for dispatching java runnables to Gecko event targets. r=snorp

Comment 2

[Chris Peterson [:cpeterson]]

Aaron says he will be working on this bug in November.

Comment 3

[Chris Peterson [:cpeterson]]

Correction: Bobby wrote this patch. Aaron says he's just submitting the patch for review.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/efc82f89a7c7
Add support for dispatching java runnables to Gecko event targets. r=snorp

Comment 5

[Cristian Brindusan [:cbrindusan]]

Backed out changeset efc82f89a7c7 (Bug 1562761) for linting failure on XPCOMEventTarget.java.

Backout: https://hg.mozilla.org/integration/autoland/rev/0e3ae20248b931d87b67c344065a75a5792a0afe

Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&revision=efc82f89a7c74dc8efe498342f78a4b379acabd4&selectedJob=275357969

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=275357969&repo=autoland&lineNumber=1102

Comment 6

[(No longer employed by Mozilla) Aaron Klotz]

I'll take it.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e552592bceae
Add support for dispatching java runnables to Gecko event targets. r=snorp

Comment 8

[Chris Peterson [:cpeterson]]

firefox71=wontfix because we don't need to uplift this runnable fix to GeckoView Beta (71).

Comment 9

[Cristina Coroiu [:ccoroiu]]

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