Skip to content
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

Use valid mimetype for HEVC/H.264/AV1 and add hev1/avc3 mimetypes #1688

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

StaZhu
Copy link
Contributor

@StaZhu StaZhu commented Dec 4, 2024

Based on conversation with Chrome Media Team, we should only support valid mimetype for future new codecs like HEVC.

For HEVC/H.264/AV1, it is recommend to use valid mimetype instead of just hvc1, avc1 or av01.

There are some limitation for avc1 / hvc1 as they only allow "out of band parameter sets", so if capture device can produce dynamic resolution video stream, only avc3 / hev1 can satisfy the needs as they supports "in band parameter sets".

Chromium may support hev1 and avc3 in the future, so also added these mimetypes for future testing.

Based on conversation with Chrome Media Team, we should only
support valid mimetype for future new codecs like HEVC.

For HEVC/H.264/AV1, it is recommend to use valid mimetype instead
of just `hvc1`, `avc1` or `av01`.

There are some limitation for `avc1` / `hvc1` as they only allow
"out of band parameter sets", so if capture device can produce
dynamic resolution video stream, only `avc3` / `hev1` can satisfy
the needs as they supports "in band parameter sets".

Chromium may support `hev1` and `avc3` in the future, so also added
these mimetypes for future testing.
@StaZhu
Copy link
Contributor Author

StaZhu commented Dec 4, 2024

cc @handellm

Copy link
Collaborator

@fippo fippo left a comment

Choose a reason for hiding this comment

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

LGTM!

@fippo fippo merged commit 45187d9 into webrtc:gh-pages Dec 5, 2024
3 checks 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.

2 participants