Capture fails after a restart of the camera

Describe the bug Capture doesn’t work after stopping and restarting the camera in release 2.7.0. An error is not generated and the picture taken listener is not invoked. This showed up in this latest release (2.7.0), it doesn’t seem to occur on 2.6.1.

To Reproduce Steps to reproduce the behavior:

1. Start camera using start
2. Stop camera using stop(false)
3. Start camera again using start
4. Call capture and the dimming UI animation occurs but addPictureTakenListener or addCameraErrorListener are never called

Expected behavior Calling capture should call either the error listener or the picture taken listener.

Using ProGuard

• with ProGuard
• without ProGuard

Device (please complete the following information):

• Device: S8+
• OS: 8.0.0
• API: 27
• App version 0.1

Branch Name (if using as a local module)

• master
• development
• release
• other (please specify)

[pvasa]

I also do see this in the debugger: kotlinx.coroutines.JobCancellationException: Job was cancelled; job=JobImpl{Cancelled}@d922855

@arvinkx good catch that was the exact issue. When a scope is created using CoroutineScope(coroutineContext) function, it attaches the scope to a new Job context. So when coroutineScope.cooutineContext.cancel() is called it cancels the job and thus no more coroutines can be created using that scope.

[arvinkx]

@pvasa Not really familiar with Kotlin but I believe the issue is related to this block of code in CameraView.kt:

override fun onPictureTaken(imageData: ByteArray) {
    coroutineScope.launch {
        pictureTakenListeners.forEach {
               it(imageData)
           }
      }
}

Once stop is called without removing the listeners, starting the camera again and calling capture in debug mode in the sample app, gets to the line with pictureTakenListeners but doesn’t call the block inside launch when breakpoints are set. Hope this helps as I’m not really familiar enough with Kotlin to know why that block is not being called. I also do see this in the debugger: kotlinx.coroutines.JobCancellationException: Job was cancelled; job=JobImpl{Cancelled}@d922855