Skip to content

Conversation

camsim99
Copy link
Contributor

@camsim99 camsim99 commented Apr 30, 2024

Breaking change: Changes default Android implementation from camera_android to camera_android_camerax via a breaking change and updates relevant documentation.

Note: Planning to land this after #6608, as it is a semi-crucial fix. Done :)

Pre-launch Checklist

auto-submit bot pushed a commit that referenced this pull request May 6, 2024
… used on Android (#6676)

Adds empty implementation for the camera plugin method `prepareForVideoRecording` that is not used on Android.

Prequel to #6629
@camsim99
Copy link
Contributor Author

camsim99 commented May 8, 2024

WIP: There are some testing changes in here to land #6697; will delete before requesting review. removed!

@camsim99 camsim99 marked this pull request as ready for review May 10, 2024 16:53
@camsim99 camsim99 requested a review from bparrishMines as a code owner May 10, 2024 16:53
@camsim99
Copy link
Contributor Author

camsim99 commented May 10, 2024

Waiting for #6697 to land to get camera_android_camerax version 0.6.5 landed but I would like the review process to start so I can hopefully land this by EOD Monday. landed!

@camsim99 camsim99 requested a review from reidbaker May 10, 2024 17:33
auto-submit bot pushed a commit that referenced this pull request May 10, 2024
…ra_android (#6697)

Makes changes needed to land #6629. Specifically:

- Fixes timing issue with `stopVideoRecording` such that the `Future` it returns will only complete when CameraX reports that the recording is finalized (via listening for the [finalized video recording event](https://developer.android.com/reference/androidx/camera/video/VideoRecordEvent.Finalize))
- Modifies `startVideoCapturing` such that the `Future` it returns will only complete when CameraX reports that video capturing has started (via listening for the [started video recording event](https://developer.android.com/reference/androidx/camera/video/VideoRecordEvent.Start))
- Adds empty implementation and TODO for implementing `setDescriptionWhileRecording`
@camsim99 camsim99 requested a review from stuartmorgan-g May 11, 2024 17:32
Copy link
Collaborator

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

🚀 🎉

@camsim99 camsim99 added the autosubmit Merge PR when tree becomes green via auto submit App label May 13, 2024
@auto-submit auto-submit bot merged commit 9588cce into flutter:main May 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 13, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request May 13, 2024
flutter/packages@6c4482a...1412041

2024-05-13 [email protected] [in_app_purchase] Update country code Android example (flutter/packages#6722)
2024-05-13 [email protected] Roll Flutter from 1dfb46e to 1255435 (6 revisions) (flutter/packages#6723)
2024-05-13 [email protected] [camera] Change default Android implementation from `camera_android` to `camera_android_camerax` (flutter/packages#6629)
2024-05-13 [email protected] [image_picker_ios] Make all headers public for SwiftPM to keep inline with CocoaPods (flutter/packages#6707)
2024-05-12 [email protected] Roll Flutter from 2aa05c1 to 1dfb46e (3 revisions) (flutter/packages#6715)
2024-05-11 [email protected] Roll Flutter from 2bfb1b0 to 2aa05c1 (26 revisions) (flutter/packages#6713)
2024-05-10 [email protected] [quick_actions] Add Swift Package Manager support (flutter/packages#6682)
2024-05-10 [email protected] [url_launcher] Add Swift Package Manager support (flutter/packages#6677)
2024-05-10 [email protected] [Android][webview_flutter] Run integration tests on emulators running Android 34 (flutter/packages#6499)
2024-05-10 [email protected] [image_picker_android] - will fix crash on Android 12+ devices (flutter/packages#6691)
2024-05-10 [email protected] [camerax] Make fixes required to swap camera_android_camerax for camera_android (flutter/packages#6697)
2024-05-10 [email protected] Update .gitignore for Swift Package Manager (flutter/packages#6705)
2024-05-10 [email protected] [quick_actions_android] Switch to `Compat` version of `ShortcutManager` to support Google surfaces (flutter/packages#6638)
2024-05-10 [email protected] [local_auth_darwin] Adds Swift Package Manager compatibility (flutter/packages#6708)
2024-05-10 [email protected] Roll Flutter from 00f4066 to 2bfb1b0 (9 revisions) (flutter/packages#6706)
2024-05-09 [email protected] [go_router] Feat add route redirect shellroutes (#114559) (flutter/packages#6432)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
TecHaxter pushed a commit to TecHaxter/flutter_packages that referenced this pull request May 22, 2024
… used on Android (flutter#6676)

Adds empty implementation for the camera plugin method `prepareForVideoRecording` that is not used on Android.

Prequel to flutter#6629
TecHaxter pushed a commit to TecHaxter/flutter_packages that referenced this pull request May 22, 2024
…ra_android (flutter#6697)

Makes changes needed to land flutter#6629. Specifically:

- Fixes timing issue with `stopVideoRecording` such that the `Future` it returns will only complete when CameraX reports that the recording is finalized (via listening for the [finalized video recording event](https://developer.android.com/reference/androidx/camera/video/VideoRecordEvent.Finalize))
- Modifies `startVideoCapturing` such that the `Future` it returns will only complete when CameraX reports that video capturing has started (via listening for the [started video recording event](https://developer.android.com/reference/androidx/camera/video/VideoRecordEvent.Start))
- Adds empty implementation and TODO for implementing `setDescriptionWhileRecording`
TecHaxter pushed a commit to TecHaxter/flutter_packages that referenced this pull request May 22, 2024
…to `camera_android_camerax` (flutter#6629)

Breaking change: Changes default Android implementation from `camera_android` to `camera_android_camerax` via a breaking change and updates relevant documentation.

~<ins>**Note:**</ins> Planning to land this after flutter#6608, as it is a semi-crucial fix.~ Done :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: camera platform-android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants