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

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
theoretical fix
  • Loading branch information
@camsim99
camsim99 committed Oct 10, 2024
commit bbac0c3530b7be4846806ca01ca35134bcf5d7ce
11 changes: 5 additions & 6 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 @@ -9,8 +9,7 @@
import androidx.media3.common.Player;
import androidx.media3.common.VideoSize;
import androidx.media3.exoplayer.ExoPlayer;

import android.util.Log;
import java.util.Objects;

final class ExoPlayerEventListener implements Player.Listener {
private final ExoPlayer exoPlayer;
Expand Down Expand Up @@ -51,8 +50,8 @@ private void sendInitialized() {
int width = videoSize.width;
int height = videoSize.height;
if (width != 0 && height != 0) {
int rotationDegrees = videoSize.unappliedRotationDegrees;
Log.e("CAMILLE rotation degrees", Integer.toString(rotationDegrees));
int rotationDegrees = Objects.requireNonNull(exoPlayer.getVideoFormat()).rotationDegrees;

// Switch the width/height if video was taken in portrait mode
if (rotationDegrees == 90 || rotationDegrees == 270) {
width = videoSize.height;
Expand All @@ -63,10 +62,10 @@ private void sendInitialized() {
// upside-down playback for videos with rotationDegrees of 180 (other orientations work
// correctly without correction).
if (rotationDegrees == 180) {
rotationCorrection = rotationDegrees;
rotationCorrection = realRotationDegrees;
}
}
events.onInitialized(width, height, exoPlayer.getDuration(), rotationCorrection);
events.onInitialized(width, height, exoPlayer.getDuration(), rotationDegrees);
}

@Override
Expand Down