Skip to content

Add Profile 20 support in Dolby Vision configuration data - #3427

Merged
copybara-service[bot] merged 1 commit into
androidx:mainfrom
DolbyLaboratories:dlb/dv-profile20-handling/dev
Sep 30, 2026
Merged

copybara-service[bot] merged 1 commit into
androidx:mainfrom
DolbyLaboratories:dlb/dv-profile20-handling/dev

Conversation

@ybai001

@ybai001 ybai001 commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Dolby Vision profile 20.0 content is currently misidentified as standard HEVC video in Media3, causing incorrect color display during playback.

This happens because when parsing a Dolby Vision container, Media3 sets the MIME type to video/hevc upon encountering the HEVC configuration box. It is supposed to override this with video/dolby-vision after parsing the Dolby Vision configuration, but since profile 20 is not yet supported by the parser, the override never occurs — leaving the content incorrectly treated as HEVC.

This fix ensures that profile 20.0 content is rejected on devices, rather than playing back with wrong colors. It is an independent contribution that does not depend on the pending AOSP changes for full profile 20 support.

Dolby Vision profile 20.0 content is currently misidentified
as standard HEVC video in Media3, causing incorrect color
display during playback.

This happens because when parsing a Dolby Vision container,
Media3 sets the MIME type to video/hevc upon encountering
the HEVC configuration box. It is supposed to override this
with video/dolby-vision after parsing the Dolby Vision
configuration, but since profile 20 is not yet supported by
the parser, the override never occurs — leaving the content
incorrectly treated as HEVC.

This fix ensures that profile 20.0 content is rejected on
devices, rather than playing back with wrong colors. It is
an independent contribution that does not depend on the
pending AOSP changes for full profile 20 support.
codecsPrefix = "dvav";
} else if (dvProfile == 10) {
codecsPrefix = "dav1";
} else if (dvProfile == 20) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Maybe outside the scope of this PR, but seems like this whole if block could be written in a simpler way with switch (dvProfile)

@microkatz microkatz self-assigned this Sep 23, 2026
@copybara-service
copybara-service Bot merged commit 2abdbfb into androidx:main Sep 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants