-
Notifications
You must be signed in to change notification settings - Fork 405
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.
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
Error when textTrackTranscodingEnabled is set to true #836
Comments
Serializing bitmap cues is currently broken, but this test is incorrectly passing. This change makes two changes to introduce the same failure (both changes are necessary, each one alone still passes): 1. Move from Robolectric to an instrumentation test. 2. Trigger the `Bitmap` to be serialized using a file descriptor, either by calling `Bitmap.asShared` in the test when constructing the `Cue`, or constructing the `Bitmap` from a 'real' image byte array instead a 1x1 token image. Issue: #836 PiperOrigin-RevId: 585643486
Serializing bitmap cues is currently broken, but this test is incorrectly passing. This change makes two changes to introduce the same failure (both changes are necessary, each one alone still passes): 1. Move from Robolectric to an instrumentation test. 2. Trigger the `Bitmap` to be serialized using a file descriptor, either by calling `Bitmap.asShared` in the test when constructing the `Cue`, or constructing the `Bitmap` from a 'real' image byte array instead a 1x1 token image. Issue: androidx/media#836 PiperOrigin-RevId: 585643486
Thanks for testing out this new feature - and reporting that it's broken :) From this part of the stack trace, I suspect something is going wrong when serializing a bitmap subtitle (PGS is a bitmap-only format):
I thought we had a test for serializing a
I've updated the test to be more realistic in 63062a9, and disabled it for now (because it's failing). I'll work on fixing the logic and re-enabling the test in a follow-up change. |
The serializable form is used when we need to serialize the result into bytes in the sample queue. The binder-based (ultimately filedescriptor-based) form is used for session/controller IPC, in order to avoid sending the bitmap bytes over the IPC. Issue: #836 #minor-release PiperOrigin-RevId: 588420836
The serializable form is used when we need to serialize the result into bytes in the sample queue. The binder-based (ultimately filedescriptor-based) form is used for session/controller IPC, in order to avoid sending the bitmap bytes over the IPC. Issue: androidx/media#836 #minor-release PiperOrigin-RevId: 588420836
Version
Media3 1.2.0
More version details
No response
Devices that reproduce the issue
Any device
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Yes
Reproduction steps
When playing a mkv video with embedded subtitles and with the
setTextTrackTranscodingEnabled
feature enabled an exception occurrs when trying to play the video.Without the feature enabled the file plays fine.
Expected result
Video should play fine with
setTextTrackTranscodingEnabled
feature enabledActual result
Media
Bug Report
adb bugreport
to android-media-github@google.com after filing this issue.The text was updated successfully, but these errors were encountered: