Rotation metadata missing when recording consecutive videos [410312131]

Fixed

Bug

Status Update

No update yet.

Description

sh...@gmail.com

created issue #1

Apr 13, 2025 10:23AM

CAMERAX VERSION: 1.5.0-alpha04

CAMERA APPLICATION NAME AND VERSION: GrapheneOS v83 (latest)

ANDROID OS BUILD NUMBER: BP1A.250305 (Android 15)

DEVICE NAME: Pixel 7A

DESCRIPTION:

Recording multiple videos consecutively causes the alternate videos to not have the rotation tag to be attached to the video. (https://github.com/user-attachments/assets/25898f13-3f03-4e41-a3cf-b271d66cb557)

STEPS TO REPRODUCE:

Orient the device horizontally
Open the GrapheneOS Camera app
Select "Video" mode
Press the "record" button 6 times, approximately two seconds apart.

OBSERVED RESULTS:

First video recorded as horizontal, second video recorded as portrait, third video recorded as horizontal.

EXPECTED RESULTS:

3 videos recorded in horizontal (landscape) orientation.

REPRODUCIBILITY: 100%

ADDITIONAL INFORMATION:

The rotation tag being missing altogether instead of being wrongly set for unknown reason.

The mediainfo metadata dump of the first video -

$ mediainfo VID_20250412_202659.mp4
General
Complete name                            : VID_20250412_202659.mp4
Format                                   : MPEG-4
Format profile                           : Base Media / Version 2
Codec ID                                 : mp42 (isom/mp42)
File size                                : 5.88 MiB
Duration                                 : 3 s 49 ms
Overall bit rate                         : 16.2 Mb/s
Frame rate                               : 29.868 FPS
Encoded date                             : 2025-04-12 10:27:02 UTC
Tagged date                              : 2025-04-12 10:27:02 UTC
xyz                                      : [redacted]
com.android.version                      : 15

Video
ID                                       : 1
Format                                   : AVC
Format/Info                              : Advanced Video Codec
Format profile                           : High@L4
Format settings                          : CABAC / 1 Ref Frames
Format settings, CABAC                   : Yes
Format settings, Reference frames        : 1 frame
Codec ID                                 : avc1
Codec ID/Info                            : Advanced Video Coding
Duration                                 : 3 s 49 ms
Source duration                          : 2 s 980 ms
Bit rate                                 : 16.4 Mb/s
Width                                    : 1 080 pixels
Height                                   : 1 920 pixels
Display aspect ratio                     : 0.562
Frame rate mode                          : Variable
Frame rate                               : 29.868 FPS
Minimum frame rate                       : 29.772 FPS
Maximum frame rate                       : 29.871 FPS
Color space                              : YUV
Chroma subsampling                       : 4:2:0
Bit depth                                : 8 bits
Scan type                                : Progressive
Bits/(Pixel*Frame)                       : 0.266
Stream size                              : 5.84 MiB (99%)
Source stream size                       : 5.84 MiB (99%)
Title                                    : VideoHandle
Language                                 : English
Encoded date                             : 2025-04-12 10:27:02 UTC
Tagged date                              : 2025-04-12 10:27:02 UTC
Color range                              : Limited
Color primaries                          : BT.709
Transfer characteristics                 : BT.709
Matrix coefficients                      : BT.709
mdhd_Duration                            : 3049
Codec configuration box                  : avcC

The mediainfo metadata dump of the second video -

$ mediainfo VID_20250412_202659.mp4
General
Complete name                            : VID_20250412_202659.mp4
Format                                   : MPEG-4
Format profile                           : Base Media / Version 2
Codec ID                                 : mp42 (isom/mp42)
File size                                : 5.88 MiB
Duration                                 : 3 s 49 ms
Overall bit rate                         : 16.2 Mb/s
Frame rate                               : 29.868 FPS
Encoded date                             : 2025-04-12 10:27:02 UTC
Tagged date                              : 2025-04-12 10:27:02 UTC
xyz                                      : [redacted]
com.android.version                      : 15

Video
ID                                       : 1
Format                                   : AVC
Format/Info                              : Advanced Video Codec
Format profile                           : High@L4
Format settings                          : CABAC / 1 Ref Frames
Format settings, CABAC                   : Yes
Format settings, Reference frames        : 1 frame
Codec ID                                 : avc1
Codec ID/Info                            : Advanced Video Coding
Duration                                 : 3 s 49 ms
Source duration                          : 2 s 980 ms
Bit rate                                 : 16.4 Mb/s
Width                                    : 1 080 pixels
Height                                   : 1 920 pixels
Display aspect ratio                     : 0.562
Frame rate mode                          : Variable
Frame rate                               : 29.868 FPS
Minimum frame rate                       : 29.772 FPS
Maximum frame rate                       : 29.871 FPS
Color space                              : YUV
Chroma subsampling                       : 4:2:0
Bit depth                                : 8 bits
Scan type                                : Progressive
Bits/(Pixel*Frame)                       : 0.266
Stream size                              : 5.84 MiB (99%)
Source stream size                       : 5.84 MiB (99%)
Title                                    : VideoHandle
Language                                 : English
Encoded date                             : 2025-04-12 10:27:02 UTC
Tagged date                              : 2025-04-12 10:27:02 UTC
Color range                              : Limited
Color primaries                          : BT.709
Transfer characteristics                 : BT.709
Matrix coefficients                      : BT.709
mdhd_Duration                            : 3049
Codec configuration box                  : avcC

Note: The details mentioned here have been originally described by a different reporter on this thread https://github.com/GrapheneOS/Camera/issues/534

Please do let us know if any additional information or clarification is needed.

Thanks a lot for your time and support!

Regards,

Mohit

Sun Apr 13 2025 15:26:13 GMT+0530 (India Standard Time).png

1.4 MB

View

Download

Sun Apr 13 2025 15:26:24 GMT+0530 (India Standard Time).png

59 KB

View

Download

Comments

sh...@gmail.com <sh...@gmail.com> #2Apr 13, 2025 12:49PM

I don't know what's going on with the link - I can't edit, it's supposed to be: https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:biometric/biometric/src/main/java/androidx/biometric/BiometricFragment.java;l=379-383;drc=e09aec149238b29f9f828fdf25280e29e245d2b1

tr_startRecording_logs.txt

243 KB

View

Download

sc...@google.com <sc...@google.com> Apr 14, 2025 01:31AM

Reassigned to le...@google.com.

le...@google.com <le...@google.com> #3Apr 14, 2025 07:05AM

Project: platform/frameworks/support
Branch: androidx-main
Author: Hao Dong <spdonghao@google.com>
Link: https://android-review.googlesource.com/3529473

Apply workaround of forcing strong bio to combined authenticators.

Expand for full commit details

Apply workaround of forcing strong bio to combined authenticators. 
 
Bug: 257670132 
Test: ./gradlew :biometric:biometric:test 
 
Change-Id: Ibb8532b5210407cfb530020f67d77210f58cfbc7

Files:

M biometric/biometric/src/main/java/androidx/biometric/AuthenticatorUtils.java
M biometric/biometric/src/main/java/androidx/biometric/BiometricViewModel.java

Hash: 90159701331b69ec3d0977c37ca3600e1f08e41b
Date: Thu Mar 06 01:19:14 2025

sh...@gmail.com <sh...@gmail.com> #4Apr 14, 2025 10:15PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Hao Dong <spdonghao@google.com>
Link: https://android-review.googlesource.com/3537000

Fix workaround of forcing strong biometrics

Expand for full commit details

Fix workaround of forcing strong biometrics 
 
1. Use mPromptInfo.getAllowedAuthenticators() to decide whether to use 
   fake crypto object to force strong biometric, instead of the 
   consolidated authenticators. 
2. Replace isStrongBiometricAllowed() with isAtLeastStrength(). 
 
Bug: 402158770 
Bug: 257670132 
Test: ./gradlew :biometric:biometric:test 
Test: manually check b/402158770 on integration test app 
 
Change-Id: I5cfb3b659781a1570ccd6aa607df3da61a559ebf

Files:

M biometric/biometric/src/main/java/androidx/biometric/AuthenticatorUtils.java
M biometric/biometric/src/main/java/androidx/biometric/BiometricManager.java
M biometric/biometric/src/main/java/androidx/biometric/BiometricViewModel.java
M biometric/biometric/src/test/java/androidx/biometric/AuthenticatorUtilsTest.java

Hash: cb2115ab72c5c98b53d26fb33a2c0bbc153d6aea
Date: Tue Mar 11 22:35:37 2025

le...@google.com <le...@google.com> #5Apr 15, 2025 04:16AM

Marked as fixed.

Hi Mohit,

Thanks for the fix. I will close this issue and create another one to track the issue in CameraX.

The issue link https://issuetracker.google.com/410692713 in case you are interested in.

Issue 410312131

Description

Issue summary

Comments

sh...@gmail.com <sh...@gmail.com> #2Apr 13, 2025 12:49PM

sc...@google.com <sc...@google.com> Apr 14, 2025 01:31AM

le...@google.com <le...@google.com> #3Apr 14, 2025 07:05AM

sh...@gmail.com <sh...@gmail.com> #4Apr 14, 2025 10:15PM

le...@google.com <le...@google.com> #5Apr 15, 2025 04:16AM

Add comment

Issue metadata