Skip to content

Commit eeb1619

Browse files
Merge branch 'main' into maps-android-remove-legacy-tests
2 parents 7af85c3 + 48c9498 commit eeb1619

10 files changed

Lines changed: 43 additions & 10 deletions

File tree

.ci/flutter_stable.version

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
6fba2447e95c451518584c35e25f5433f14d888c
1+
fcf2c11572af6f390246c056bc905eca609533a0

packages/camera/camera_android_camerax/CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
## 0.6.19
2+
3+
* Changes target rotation of captured images to current default display rotation to fix captured
4+
photo orientation to upright.
5+
16
## 0.6.18+3
27

38
* Fixes incorrect camera preview mirroring for front cameras of devices using the Impeller backend.

packages/camera/camera_android_camerax/lib/src/android_camera_camerax.dart

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -392,7 +392,8 @@ class AndroidCameraCameraX extends CameraPlatform {
392392
// Configure ImageCapture instance.
393393
imageCapture = proxy.newImageCapture(
394394
resolutionSelector: presetResolutionSelector,
395-
/* use CameraX default target rotation */ targetRotation: null,
395+
/* use CameraX default target rotation */ targetRotation:
396+
await deviceOrientationManager.getDefaultDisplayRotation(),
396397
);
397398

398399
// Configure ImageAnalysis instance.
@@ -966,9 +967,9 @@ class AndroidCameraCameraX extends CameraPlatform {
966967
await imageCapture!.setFlashMode(CameraXFlashMode.off);
967968
}
968969

969-
// Set target rotation to default CameraX rotation only if capture
970-
// orientation not locked.
971-
if (!captureOrientationLocked && shouldSetDefaultRotation) {
970+
// Set target rotation to the current default CameraX rotation if
971+
// the capture orientation is not locked.
972+
if (!captureOrientationLocked) {
972973
await imageCapture!.setTargetRotation(
973974
await deviceOrientationManager.getDefaultDisplayRotation(),
974975
);

packages/camera/camera_android_camerax/pubspec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: camera_android_camerax
22
description: Android implementation of the camera plugin using the CameraX library.
33
repository: https://github.com/flutter/packages/tree/main/packages/camera/camera_android_camerax
44
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+camera%22
5-
version: 0.6.18+3
5+
version: 0.6.19
66

77
environment:
88
sdk: ^3.7.0

packages/camera/camera_android_camerax/test/android_camera_camerax_test.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3488,7 +3488,7 @@ void main() {
34883488
);
34893489

34903490
test(
3491-
'takePicture sets ImageCapture target rotation to currrent photo rotation when orientation unlocked',
3491+
'takePicture sets ImageCapture target rotation as expected when orientation locked or unlocked',
34923492
() async {
34933493
final AndroidCameraCameraX camera = AndroidCameraCameraX();
34943494
final MockImageCapture mockImageCapture = MockImageCapture();
@@ -3531,7 +3531,7 @@ void main() {
35313531
// Orientation is unlocked and plugin does not need to set default target
35323532
// rotation manually.
35333533
await camera.takePicture(cameraId);
3534-
verifyNever(mockImageCapture.setTargetRotation(any));
3534+
verify(mockImageCapture.setTargetRotation(defaultTargetRotation));
35353535

35363536
// Orientation is locked and plugin does not need to set default target
35373537
// rotation manually.

packages/in_app_purchase/in_app_purchase_android/example/integration_test/in_app_purchase_test.dart

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,8 @@ void main() {
106106
testWidgets('BillingClient.queryPurchaseHistory',
107107
(WidgetTester tester) async {
108108
try {
109+
// Intentional use of a deprecated method to make sure it still works.
110+
// ignore: deprecated_member_use
109111
await billingClient.queryPurchaseHistory(ProductType.inapp);
110112
} on MissingPluginException {
111113
fail('Method channel is not setup correctly');

packages/video_player/video_player_platform_interface/CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
## NEXT
1+
## 6.4.0
22

3+
* Adds HTML5 video poster support as a VideoPlayerWebOptions.
34
* Updates minimum supported SDK version to Flutter 3.27/Dart 3.6.
45

56
## 6.3.0

packages/video_player/video_player_platform_interface/lib/video_player_platform_interface.dart

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -429,6 +429,7 @@ class VideoPlayerWebOptions {
429429
this.controls = const VideoPlayerWebOptionsControls.disabled(),
430430
this.allowContextMenu = true,
431431
this.allowRemotePlayback = true,
432+
this.poster,
432433
});
433434

434435
/// Additional settings for how control options are displayed
@@ -439,6 +440,9 @@ class VideoPlayerWebOptions {
439440

440441
/// Whether remote playback is allowed
441442
final bool allowRemotePlayback;
443+
444+
/// The URL of the poster image to be displayed before the video starts
445+
final Uri? poster;
442446
}
443447

444448
/// [VideoPlayerWebOptions] can be used to set how control options are displayed

packages/video_player/video_player_platform_interface/pubspec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ repository: https://github.com/flutter/packages/tree/main/packages/video_player/
44
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+video_player%22
55
# NOTE: We strongly prefer non-breaking changes, even at the expense of a
66
# less-clean API. See https://flutter.dev/go/platform-interface-breaking-changes
7-
version: 6.3.0
7+
version: 6.4.0
88

99
environment:
1010
sdk: ^3.6.0

packages/video_player/video_player_platform_interface/test/video_player_web_options_test.dart

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,4 +29,24 @@ void main() {
2929
expect(options.allowRemotePlayback, isTrue);
3030
},
3131
);
32+
33+
group('VideoPlayerOptions poster', () {
34+
test(
35+
'defaults to null',
36+
() {
37+
const VideoPlayerWebOptions options = VideoPlayerWebOptions();
38+
expect(options.poster, null);
39+
},
40+
);
41+
42+
test(
43+
'with a value',
44+
() {
45+
final VideoPlayerWebOptions options = VideoPlayerWebOptions(
46+
poster: Uri.parse('https://example.com/poster.jpg'),
47+
);
48+
expect(options.poster, Uri.parse('https://example.com/poster.jpg'));
49+
},
50+
);
51+
});
3252
}

0 commit comments

Comments
 (0)