Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix CTS extension errors"

Browse Source
This commit is contained in:
Ravneet Dhanjal
2023-01-06 21:14:47 +00:00
committed by Android (Google) Code Review
parent 18c7a36302 b7fb257b4b
commit 251c756387
2 changed files with 26 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
core/java/android/hardware/camera2/impl/CameraAdvancedExtensionSessionImpl.java
View File

@@ -176,7 +176,7 @@ public final class CameraAdvancedExtensionSessionImpl extends CameraExtensionSes
}
Surface postviewSurface = null;
if (burstCaptureSurface != null) {
if (burstCaptureSurface != null && config.getPostviewOutputConfiguration() != null) {
CameraExtensionUtils.SurfaceInfo burstCaptureSurfaceInfo =
CameraExtensionUtils.querySurface(burstCaptureSurface);
Size burstCaptureSurfaceSize =
@@ -193,8 +193,7 @@ public final class CameraAdvancedExtensionSessionImpl extends CameraExtensionSes
postviewSurface = CameraExtensionUtils.getPostviewSurface(
config.getPostviewOutputConfiguration(), supportedPostviewSizes,
burstCaptureSurfaceInfo.mFormat);
if ((config.getPostviewOutputConfiguration() != null) && (postviewSurface == null)) {
if (postviewSurface == null) {
throw new IllegalArgumentException("Unsupported output surface for postview!");
}
}
@@ -456,11 +455,17 @@ public final class CameraAdvancedExtensionSessionImpl extends CameraExtensionSes
}
private void validateCaptureRequestTargets(@NonNull CaptureRequest request) {
if ((request.getTargets().size() == 1) &&
(!request.containsTarget(mClientRepeatingRequestSurface) ||
!request.containsTarget(mClientCaptureSurface))) {
throw new IllegalArgumentException("Target output combination requested is " +
"not supported!");
if (request.getTargets().size() == 1) {
boolean containsCaptureTarget =
mClientCaptureSurface != null && request.containsTarget(mClientCaptureSurface);
boolean containsRepeatingTarget =
mClientRepeatingRequestSurface != null &&
request.containsTarget(mClientRepeatingRequestSurface);
if (!containsCaptureTarget && !containsRepeatingTarget) {
throw new IllegalArgumentException("Target output combination requested is " +
"not supported!");
}
}
if ((request.getTargets().size() == 2) &&

20
core/java/android/hardware/camera2/impl/CameraExtensionSessionImpl.java
View File

@@ -200,7 +200,7 @@ public final class CameraExtensionSessionImpl extends CameraExtensionSession {
}
Surface postviewSurface = null;
if (burstCaptureSurface != null) {
if (burstCaptureSurface != null && config.getPostviewOutputConfiguration() != null) {
CameraExtensionUtils.SurfaceInfo burstCaptureSurfaceInfo =
CameraExtensionUtils.querySurface(burstCaptureSurface);
Size burstCaptureSurfaceSize =
@@ -218,7 +218,7 @@ public final class CameraExtensionSessionImpl extends CameraExtensionSession {
config.getPostviewOutputConfiguration(), supportedPostviewSizes,
burstCaptureSurfaceInfo.mFormat);
if ((config.getPostviewOutputConfiguration() != null) && (postviewSurface == null)) {
if (postviewSurface == null) {
throw new IllegalArgumentException("Unsupported output surface for postview!");
}
}
@@ -717,11 +717,17 @@ public final class CameraExtensionSessionImpl extends CameraExtensionSession {
}
private void validateCaptureRequestTargets(@NonNull CaptureRequest request) {
if ((request.getTargets().size() == 1) &&
(!request.containsTarget(mClientRepeatingRequestSurface) ||
!request.containsTarget(mClientCaptureSurface))) {
throw new IllegalArgumentException("Target output combination requested is " +
"not supported!");
if (request.getTargets().size() == 1) {
boolean containsCaptureTarget =
mClientCaptureSurface != null && request.containsTarget(mClientCaptureSurface);
boolean containsRepeatingTarget =
mClientRepeatingRequestSurface != null &&
request.containsTarget(mClientRepeatingRequestSurface);
if (!containsCaptureTarget && !containsRepeatingTarget) {
throw new IllegalArgumentException("Target output combination requested is " +
"not supported!");
}
}
if ((request.getTargets().size() == 2) &&