GeckoView socket process crashes on startup
Categories
(GeckoView :: Sandboxing, defect, P1)
Tracking
(firefox74 fixed)
Tracking | Status | |
---|---|---|
firefox74 | --- | fixed |
People
(Reporter: bdahl, Assigned: bdahl)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Running a mochitest locally fails. The socket process crashes with the following warning
WARNING: NS_ENSURE_TRUE(appShell) failed
The reason the appshell is blocked is because we don't specify which process that appshell can be run in for android. This make is default to any process, but for sockets you you have to explicitly enable it e.g. ProcessSelector.ALLOW_IN_GPU_RDD_VR_AND_SOCKET_PROCESS
Assignee | ||
Comment 1•5 years ago
|
||
Hey Bryon,
I can fix the socket process on android with above fix. However, I'm wondering if we should just disable for now? Also, is the socket process being actively worked on?
Comment 2•5 years ago
|
||
Socket process is being used/worked on, yes. We hope to make it the default mode of operation, so let's try to make it work here.
In your test, is e10s being used?
Updated•5 years ago
|
Assignee | ||
Comment 3•5 years ago
|
||
In local Android builds the socket process is enabled. However, AppShell
creation is not allowed which causes the process to crash. This syncs the
flags with desktop.
Assignee | ||
Comment 4•5 years ago
|
||
(In reply to Byron Campen [:bwc] from comment #2)
In your test, is e10s being used?
Yeah, mochitests on android have e10s enabled.
One thing that's problematic here though, is local mochitest and mochitest run on the try server have different settings for network.process.enabled
. I don't quite understand the discrepancy (maybe this or this). I think we should work to unify this though, so we don't run into the situation again where things fail locally, but not on try.
Comment 7•5 years ago
|
||
bugherder |
Assignee | ||
Updated•5 years ago
|
Comment 8•2 years ago
|
||
Moving socket process bugs to the new GeckoView::Sandboxing component.
Description
•