commit cdaafb35c2c6be33bdbcb506948c90284d2687e9
author Emilian Peev <epeev@google.com> Wed Jul 26 22:39:33 2023 +0000
committer Emilian Peev <epeev@google.com> Thu Jul 27 16:05:13 2023 +0000
tree 57099b2e04343aab1d2684fc78ac74d5ee449e27
parent d5da53c6aac1cdb67064d59e5517e6c7b1da20fc

Camera: Handle single logical requests when notifying requests

Consider cases where clients pass single logical capture requests
to physical outputs and notify request frame number listeners
accordingly.

Bug: 292144312
Test: Camera CTS
Change-Id: I3a44cc9a7eaa5b2107f494bddb5e80706eec2bfd

services/camera/libcameraservice/device3/Camera3Device.cpp

diff --git a/services/camera/libcameraservice/device3/Camera3Device.cpp b/services/camera/libcameraservice/device3/Camera3Device.cpp
index 52ba782..a275cfb 100644
--- a/services/camera/libcameraservice/device3/Camera3Device.cpp
+++ b/services/camera/libcameraservice/device3/Camera3Device.cpp
@@ -4014,8 +4014,11 @@
                 sp<Camera3Device> parent = mParent.promote();
                 if (parent != nullptr) {
                     const std::string& streamCameraId = outputStream->getPhysicalCameraId();
+                    // Consider the case where clients are sending a single logical camera request
+                    // to physical output/outputs
+                    bool singleRequest = captureRequest->mSettingsList.size() == 1;
                     for (const auto& settings : captureRequest->mSettingsList) {
-                        if ((streamCameraId.empty() &&
+                        if (((streamCameraId.empty() || singleRequest) &&
                                 parent->getId() == settings.cameraId) ||
                                 streamCameraId == settings.cameraId) {
                             outputStream->fireBufferRequestForFrameNumber(