1522890 - GeckoSystemStateListener is incompatible with isolatedProcess

Status: RESOLVED FIXED

(Core :: Security: Process Sandboxing, defect, P1)

Description

[Gian-Carlo Pascutto [:gcp]]

Registering Observers on the Application context, as far as I can tell to see changes to the system animation settings and connected input devices. Can't run in the Gecko process if we want sandboxing.

E/GeckoCrashHandler( 9968): >>> REPORTING UNCAUGHT EXCEPTION FROM THREAD 25817 ("Gecko")
E/GeckoCrashHandler( 9968): java.lang.NullPointerException: Attempt to invoke interface method 'void android.content.IContentService.registerContentObserver(android.net.Uri, boolean, android.database.IContentObserver, int)' on a null object reference
E/GeckoCrashHandler( 9968): at android.content.ContentResolver.registerContentObserver(ContentResolver.java:1571)
E/GeckoCrashHandler( 9968): at android.content.ContentResolver.registerContentObserver(ContentResolver.java:1563)
E/GeckoCrashHandler( 9968): at org.mozilla.gecko.GeckoSystemStateListener.initialize(GeckoSystemStateListener.java:60)
E/GeckoCrashHandler( 9968): at org.mozilla.gecko.GeckoThread.initGeckoEnvironment(GeckoThread.java:286)
E/GeckoCrashHandler( 9968): at org.mozilla.gecko.GeckoThread.run(GeckoThread.java:416)
E/GeckoCrashHandler( 9968): Main thread (1) stack:
E/GeckoCrashHandler( 9968): android.os.MessageQueue.nativePollOnce(Native Method)
E/GeckoCrashHandler( 9968): android.os.MessageQueue.next(MessageQueue.java:143)
E/GeckoCrashHandler( 9968): android.os.Looper.loop(Looper.java:122)
E/GeckoCrashHandler( 9968): android.app.ActivityThread.main(ActivityThread.java:5254)
E/GeckoCrashHandler( 9968): java.lang.reflect.Method.invoke(Native Method)
E/GeckoCrashHandler( 9968): java.lang.reflect.Method.invoke(Method.java:372)
E/GeckoCrashHandler( 9968): com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:903)
E/GeckoCrashHandler( 9968): com.android.internal.os.ZygoteInit.main(ZygoteInit.java:698)
E/art ( 9968): No implementation found for void org.mozilla.gecko.GeckoAppShell.reportJavaCrash(java.lang.Throwable, java.lan

Comment 1

[Gian-Carlo Pascutto [:gcp]]

I have one test failure which I think is related to having to disable this feature:
layout/style/test/test_mq_prefers_reduced_motion_dynamic.html

Comment 2

[[:owlish] 🦉 PST]

There was a mistake in me naming the branch, so this is for this bug:

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cd5376dec81c
Don't initialize GeckoViewStateListener in GeckoThread if on a child process r=geckoview-reviewers,aklotz

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