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

webcodecs: Temporal SVC encode #28063

Merged
merged 1 commit into from Mar 16, 2021

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Mar 12, 2021

  1. Adding VideoEncoderConfig.scalabilityMode and EncodedVideoChunkMetadata
  2. Connecting them to SVC fields in media::VideoEncoder config and output
  3. Renaming VideoEncoderOutputCallback to EncodedVideoChunkOutputCallback
    in order to comply with the spec.
  4. Deleting EncodedVideoMetadata, it doesn't do anything,
    and can be confused with EncodedVideoChunkMetadata
  5. New test: temporal_svc.any.js

Bug: 1176474
Change-Id: Ia89ec093c55df4495f50d51262bcc75c578fd8d6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2755947
Commit-Queue: Eugene Zemtsov <eugene@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Reviewed-by: Chrome Cunningham <chcunningham@chromium.org>
Cr-Commit-Position: refs/heads/master@{#863171}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-2755947 branch 2 times, most recently from 529e12d to 42f2176 Compare March 16, 2021 02:06
1. Adding VideoEncoderConfig.scalabilityMode and EncodedVideoChunkMetadata
2. Connecting them to SVC fields in media::VideoEncoder config and output
3. Renaming VideoEncoderOutputCallback to EncodedVideoChunkOutputCallback
   in order to comply with the spec.
4. Deleting EncodedVideoMetadata, it doesn't do anything,
   and can be confused with EncodedVideoChunkMetadata
5. New test: temporal_svc.any.js

Bug: 1176474
Change-Id: Ia89ec093c55df4495f50d51262bcc75c578fd8d6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2755947
Commit-Queue: Eugene Zemtsov <eugene@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Reviewed-by: Chrome Cunningham <chcunningham@chromium.org>
Cr-Commit-Position: refs/heads/master@{#863171}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants