RejectedExecutionException originating from VideoCapture.clearPipeline() [273713906]

Fixed

Bug

Status Update

No update yet.

Description

ri...@gmail.com

created issue #1

Mar 15, 2023 03:51PM

CAMERAX VERSION: 1.3.0-alpha03

CAMERA APPLICATION NAME AND VERSION: LinkedIn 4.1.793

ANDROID OS BUILD NUMBER: 8, 10, 11, 12, 13

DEVICE NAME: Several Samsung (A51, A11, A32..) and Xiaomi (Redmi 9) devices

DESCRIPTION:

Our crash monitoring reports a few instances of the following crash:

java.util.concurrent.RejectedExecutionException:Handler (android.os Handler) {85a71a1} is shutting down()
at androidx.camera.core.impl.utils.executor.HandlerScheduledExecutorService . HandlerScheduledExecutorService . createPostFailedException() (HandlerScheduledExecutorService.java:184)
at androidx.camera.core.impl.utils.executor.HandlerScheduledExecutorService . HandlerScheduledExecutorService . execute() (HandlerScheduledExecutorService.java:179)
at androidx.camera.core.processing.DefaultSurfaceProcessor . DefaultSurfaceProcessor . release() (DefaultSurfaceProcessor.java:148)
at androidx.camera.core.processing.SurfaceProcessorNode . SurfaceProcessorNode . release() (SurfaceProcessorNode.java:256)
at androidx.camera.video.VideoCapture . VideoCapture . clearPipeline() (VideoCapture.java:558)
at androidx.camera.video.VideoCapture . VideoCapture . onStateDetached() (VideoCapture.java:346)
at androidx.camera.camera2.internal.Camera2CameraImpl . Camera2CameraImpl . notifyStateDetachedToUseCases() (Camera2CameraImpl.java:821)
at androidx.camera.camera2.internal.Camera2CameraImpl . Camera2CameraImpl . detachUseCases() (Camera2CameraImpl.java:840)
at androidx.camera.core.internal.CameraUseCaseAdapter . CameraUseCaseAdapter . detachUseCases() (CameraUseCaseAdapter.java:366)
at androidx.camera.lifecycle.LifecycleCamera . LifecycleCamera . onStop() (LifecycleCamera.java:97)
at androidx.camera.lifecycle.LifecycleCamera . LifecycleCamera . suspend() (LifecycleCamera.java:140)
at androidx.camera.lifecycle.LifecycleCameraRepository . LifecycleCameraRepository . suspendUseCases() (LifecycleCameraRepository.java:449)
at androidx.camera.lifecycle.LifecycleCameraRepository . LifecycleCameraRepository . setInactive() (LifecycleCameraRepository.java:394)
at androidx.camera.lifecycle.LifecycleCameraRepository$LifecycleCameraRepositoryObserver . LifecycleCameraRepository . onStop() (LifecycleCameraRepository.java:522)
at java.lang.reflect.Method . Method . invoke() (Method.java:-2)
at androidx.lifecycle.ClassesInfoCache$MethodReference . ClassesInfoCache . invokeCallback() (ClassesInfoCache.java:225)
at androidx.lifecycle.ClassesInfoCache$CallbackInfo . ClassesInfoCache . invokeMethodsForEvent() (ClassesInfoCache.java:199)
at androidx.lifecycle.ClassesInfoCache$CallbackInfo . ClassesInfoCache . invokeCallbacks() (ClassesInfoCache.java:190)
at androidx.lifecycle.ReflectiveGenericLifecycleObserver . ReflectiveGenericLifecycleObserver . onStateChanged() (ReflectiveGenericLifecycleObserver.java:40)
at androidx.lifecycle.LifecycleRegistry$ObserverWithState . LifecycleRegistry . dispatchEvent() (LifecycleRegistry.java:360)
at androidx.lifecycle.LifecycleRegistry . LifecycleRegistry . backwardPass() (LifecycleRegistry.java:290)
at androidx.lifecycle.LifecycleRegistry . LifecycleRegistry . sync() (LifecycleRegistry.java:308)
at androidx.lifecycle.LifecycleRegistry . LifecycleRegistry . moveToState() (LifecycleRegistry.java:151)
at androidx.lifecycle.LifecycleRegistry . LifecycleRegistry . handleLifecycleEvent() (LifecycleRegistry.java:134)
at androidx.fragment.app.Fragment . Fragment . performStop() (Fragment.java:3318)

LIST ANY EXPERIMENTAL FEATURES: androidx.camera.view.video.ExperimentalVideo

ADDITIONAL INFORMATION: 8 external instances reported over the past week, no internal reports, not reproducible during our normal testing

CODE FRAGMENTS (this will help us troubleshoot your issues):

    onResume() {
        if (controller != null) {
            return
        }
        controller = LifecycleCameraController(activity)
        // set up target size, capture quality, etc
        controller.initializationFuture.addListener({
            try {
                controller.initializationFuture.get()
                controller.bindToLifecycle(fragment)
            } catch (e: ExecutionException) {
                // report error
            } catch (e: IllegalStateException) {
                // report error
            }
        }, mainExecutor)
    }

Comments

wu...@google.com <wu...@google.com> #2Mar 17, 2023 12:29AM

seems to be dup: b/212982460

le...@google.com <le...@google.com> Mar 17, 2023 12:59AM

Reassigned to le...@google.com.

le...@google.com <le...@google.com> Mar 17, 2023 06:42AM

Accepted by le...@google.com.

ap...@google.com <ap...@google.com> #3Mar 21, 2023 02:07AM

Fundamentally the issue here is that the selection logic happening in composition depends on the text layout result which is available on measure. So we end up using stale TextLayoutResult which causes the issue.

le...@google.com <le...@google.com> #4Mar 21, 2023 10:26AM

Marked as fixed.

This is the same exception as in b/229378536 and b/221580975 as well, and it also involves selection, so I'm wondering if those two bugs have the same cause Anastasia described.

pr...@google.com <pr...@google.com> #5Apr 19, 2023 06:13PM

There is a similar problem regarding Magnifier but I don't think these are directly related. The main cause for this issue is the async relation between TextFieldValue and layoutResult. Magnifier related crashes don't seem to be caused by changes in TextFieldValue or layout.

Issue 273713906