2084934 - Enable the forked executor for crashes in Android Isolated Processes

Status: RESOLVED FIXED

(Toolkit :: Crash Reporting, enhancement)

Description

[Simon Chopin [:schopin]]

This can use a simple heuristic: the one characteristic of Isolated Processes that makes the standard ptrace-based approach fail is that they run under their own, different UID.

Comment 1

[Simon Chopin [:schopin]]

Attachment: Bug 2084934 - Use the remote executor for processes running under another UID r=#crashreporting-reviewers

The first characteristic of isolated processes that prevents our usual
way of getting crash dumps is that they run under their own UID.
Luckily, that also gives us a handy way of identifying them.

Comment 2

[Simon Chopin [:schopin]]

Attachment: Bug 2084934 - Enable content crash tests on isolated processes r=#crashreporting-reviewers

We just have to disable the assertions that rely on annotations since
they're still buggy, see bug 2082922.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/92dd8b19b278
https://hg.mozilla.org/integration/autoland/rev/d35528872e54
Use the remote executor for processes running under another UID r=crashreporting-reviewers,afranchuk
https://github.com/mozilla-firefox/firefox/commit/6e1894f5fe91
https://hg.mozilla.org/integration/autoland/rev/d4d0eac5ef53
Enable content crash tests on isolated processes r=crashreporting-reviewers,geckoview-reviewers,afranchuk,m_kato

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/d35528872e54
https://hg.mozilla.org/mozilla-central/rev/d4d0eac5ef53