1522886 - MemoryMonitor is incompatible with isolatedProcess

Status: RESOLVED INVALID

(GeckoView Graveyard :: Sandboxing, defect, P1)

Description

[Gian-Carlo Pascutto [:gcp]]

MemoryMonitor sets an observer on the ApplicationContext that listens for memory and disk pressure events. That's not possible in an isolatedProcess.

Looks like it'll have to move to the main service instead of the Gecko one where it runs now.

Comment 1

[Gian-Carlo Pascutto [:gcp]]

E/GeckoCrashHandler( 5877): java.lang.SecurityException: Isolated process not allowed to call registerReceiver
E/GeckoCrashHandler( 5877): at android.os.Parcel.readException(Parcel.java:1546)
E/GeckoCrashHandler( 5877): at android.os.Parcel.readException(Parcel.java:1499)
E/GeckoCrashHandler( 5877): at android.app.ActivityManagerProxy.registerReceiver(ActivityManagerNative.java:2782)
E/GeckoCrashHandler( 5877): at android.app.ContextImpl.registerReceiverInternal(ContextImpl.java:1655)
E/GeckoCrashHandler( 5877): at android.app.ContextImpl.registerReceiver(ContextImpl.java:1623)
E/GeckoCrashHandler( 5877): at android.app.ContextImpl.registerReceiver(ContextImpl.java:1617)
E/GeckoCrashHandler( 5877): at android.content.ContextWrapper.registerReceiver(ContextWrapper.java:489)
E/GeckoCrashHandler( 5877): at org.mozilla.gecko.MemoryMonitor.init(MemoryMonitor.java:86)
E/GeckoCrashHandler( 5877): at org.mozilla.gecko.GeckoApplication.onCreate(GeckoApplication.java:326)
E/GeckoCrashHandler( 5877): at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1012)
E/GeckoCrashHandler( 5877): at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4553)
E/GeckoCrashHandler( 5877): at android.app.ActivityThread.access$1500(ActivityThread.java:151)
E/GeckoCrashHandler( 5877): at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1364)
E/GeckoCrashHandler( 5877): at android.os.Handler.dispatchMessage(Handler.java:102)
E/GeckoCrashHandler( 5877): at android.os.Looper.loop(Looper.java:135)
E/GeckoCrashHandler( 5877): at android.app.ActivityThread.main(ActivityThread.java:5254)
E/GeckoCrashHandler( 5877): at java.lang.reflect.Method.invoke(Native Method)
E/GeckoCrashHandler( 5877): at java.lang.reflect.Method.invoke(Method.java:372)
E/GeckoCrashHandler( 5877): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:903)
E/GeckoCrashHandler( 5877): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:698)

Comment 2

[[:owlish] 🦉 PST]

There's no MemoryMonitor anymore! ʢᵕᴗᵕʡ

Comment 3

[Chris Peterson [:cpeterson]]

Moving isolated process bugs to the new GeckoView::Sandboxing component.