Merge "CameraDeviceImpl: Follow through reporting device errors even if the remote device was never initialized."

This commit is contained in:
Austin Borger
2023-01-09 21:57:19 +00:00
committed by Android (Google) Code Review

View File

@@ -85,6 +85,7 @@ public class CameraDeviceImpl extends CameraDevice
// TODO: guard every function with if (!mRemoteDevice) check (if it was closed)
private ICameraDeviceUserWrapper mRemoteDevice;
private boolean mRemoteDeviceInit = false;
// Lock to synchronize cross-thread access to device public interface
final Object mInterfaceLock = new Object(); // access from this class and Session only!
@@ -336,6 +337,8 @@ public class CameraDeviceImpl extends CameraDevice
mDeviceExecutor.execute(mCallOnOpened);
mDeviceExecutor.execute(mCallOnUnconfigured);
mRemoteDeviceInit = true;
}
}
@@ -1752,8 +1755,8 @@ public class CameraDeviceImpl extends CameraDevice
}
synchronized(mInterfaceLock) {
if (mRemoteDevice == null) {
return; // Camera already closed
if (mRemoteDevice == null && mRemoteDeviceInit) {
return; // Camera already closed, user is not interested in errors anymore.
}
// Redirect device callback to the offline session in case we are in the middle