diff --git a/packages/camera/camera_android/CHANGELOG.md b/packages/camera/camera_android/CHANGELOG.md index e809b13d6e6..ef2cc5f49fa 100644 --- a/packages/camera/camera_android/CHANGELOG.md +++ b/packages/camera/camera_android/CHANGELOG.md @@ -1,3 +1,7 @@ +## 0.10.8+12 + +* Fixes handling of autofocus state when taking a picture. + ## 0.10.8+11 * Downgrades AGP version for compatibility with legacy projects. diff --git a/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/CameraCaptureCallback.java b/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/CameraCaptureCallback.java index 4f5e43593e0..2737e80f2f5 100644 --- a/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/CameraCaptureCallback.java +++ b/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/CameraCaptureCallback.java @@ -31,7 +31,7 @@ class CameraCaptureCallback extends CaptureCallback { CaptureResult.Key aeStateKey = CaptureResult.CONTROL_AE_STATE; @VisibleForTesting @NonNull - CaptureResult.Key afStateKey = CaptureResult.CONTROL_AE_STATE; + CaptureResult.Key afStateKey = CaptureResult.CONTROL_AF_STATE; private CameraCaptureCallback( @NonNull CameraCaptureStateListener cameraStateListener, diff --git a/packages/camera/camera_android/android/src/test/java/io/flutter/plugins/camera/CameraCaptureCallbackTest.java b/packages/camera/camera_android/android/src/test/java/io/flutter/plugins/camera/CameraCaptureCallbackTest.java index 75a5b25995e..f515b359b81 100644 --- a/packages/camera/camera_android/android/src/test/java/io/flutter/plugins/camera/CameraCaptureCallbackTest.java +++ b/packages/camera/camera_android/android/src/test/java/io/flutter/plugins/camera/CameraCaptureCallbackTest.java @@ -69,4 +69,22 @@ public void onCaptureCompleted_updatesCameraCaptureProperties() { verify(mockCaptureProps, times(1)).setLastSensorExposureTime(2L); verify(mockCaptureProps, times(1)).setLastSensorSensitivity(3); } + + @Test + public void onCaptureCompleted_checksBothAutoFocusAndAutoExposure() { + CameraCaptureSession mockSession = mock(CameraCaptureSession.class); + CaptureRequest mockRequest = mock(CaptureRequest.class); + TotalCaptureResult mockResult = mock(TotalCaptureResult.class); + + cameraCaptureCallback.onCaptureCompleted(mockSession, mockRequest, mockResult); + + // This is inherently somewhat fragile since it is testing internal implementation details, + // but it is important to test that the code is actually using both of the expected states + // since it's easy to typo one of these constants as the other. Ideally this would be tested + // via the state machine output (CameraCaptureCallbackStatesTest.java), but testing the state + // machine requires overriding the keys, so can't test that the right real keys are used in + // production. + verify(mockResult, times(1)).get(CaptureResult.CONTROL_AE_STATE); + verify(mockResult, times(1)).get(CaptureResult.CONTROL_AF_STATE); + } } diff --git a/packages/camera/camera_android/pubspec.yaml b/packages/camera/camera_android/pubspec.yaml index f2c68922f17..86f3665dc08 100644 --- a/packages/camera/camera_android/pubspec.yaml +++ b/packages/camera/camera_android/pubspec.yaml @@ -3,7 +3,7 @@ description: Android implementation of the camera plugin. repository: https://github.com/flutter/packages/tree/main/packages/camera/camera_android issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+camera%22 -version: 0.10.8+11 +version: 0.10.8+12 environment: sdk: ">=2.19.0 <4.0.0"