Skip to content

[video_player_android] Correct rotation of videos recorded by the camera #7846

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
auto-submit merged 16 commits into from
Oct 21, 2024
Merged

[video_player_android] Correct rotation of videos recorded by the camera #7846

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
5b8ad64
debugging
camsim99 Oct 9, 2024
bbac0c3
theoretical fix
camsim99 Oct 10, 2024
43898cb
Mild cleanup
camsim99 Oct 10, 2024
95d5e43
Undo extra import
camsim99 Oct 10, 2024
3306352
Fix logic + start tests
camsim99 Oct 15, 2024
3524885
Merge remote-tracking branch 'upstream/main' into vp_fix
camsim99 Oct 16, 2024
2664b51
Fix code + add tests
camsim99 Oct 16, 2024
1ba7efc
Remove print statement, bump version
camsim99 Oct 16, 2024
0ff09c6
Add Impeller case
camsim99 Oct 16, 2024
a374698
Merge remote-tracking branch 'upstream/main' into vp_fix
camsim99 Oct 17, 2024
a74cea9
Cleanup
camsim99 Oct 17, 2024
da92163
Add comment
camsim99 Oct 17, 2024
1a65eea
Correct changelog
camsim99 Oct 17, 2024
ee0f7de
Address review
camsim99 Oct 18, 2024
fc22ee6
Merge remote-tracking branch 'upstream/main' into vp_fix
camsim99 Oct 18, 2024
5f2a824
address review
camsim99 Oct 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove print statement, bump version
  • Loading branch information
@camsim99
camsim99 committed Oct 16, 2024
commit 1ba7efc85dc6957d411c2e768d61262d4af6cd8d
6 changes: 6 additions & 0 deletions packages/video_player/video_player_android/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
## 2.7.14

* Changes the rotation correction calculation for Android API 21+ to use
that which is reported by the video's format instead of the unapplied
rotation degrees that Exoplayer does not report on Android API 21+.

## 2.7.13

* When `AndroidVideoPlayer` attempts to operate on a `textureId` that is not
Expand Down
2 changes: 0 additions & 2 deletions ..._android/android/src/main/java/io/flutter/plugins/videoplayer/ExoPlayerEventListener.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ private void setBuffering(boolean buffering) {

@SuppressWarnings("SuspiciousNameCombination")
private void sendInitialized() {
System.out.println("Hello???");
if (isInitialized) {
return;
}
Expand Down Expand Up @@ -76,7 +75,6 @@ private void sendInitialized() {
height = videoSize.width;
}
}

events.onInitialized(width, height, exoPlayer.getDuration(), rotationCorrection);
}

Expand Down
Binary file modified packages/video_player/video_player_android/example/assets/video.mp4
View file
Open in desktop
Binary file not shown.
2 changes: 1 addition & 1 deletion packages/video_player/video_player_android/pubspec.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: video_player_android
description: Android implementation of the video_player plugin.
repository: https://github.com/flutter/packages/tree/main/packages/video_player/video_player_android
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+video_player%22
version: 2.7.13
version: 2.7.14

environment:
sdk: ^3.5.0
Expand Down