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

webrtc: add setParameterOptions to sender.setParameters #43705

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Dec 18, 2023

adding a new RTCSetParameterOptions object which has a sequence of
RTCEncodingOptions similar to
WebCodecs VideoEncoderEncodeOptions
https://w3c.github.io/webcodecs/#dictdef-videoencoderencodeoptions
and its keyFrame flag.

On the native side, this adds the request_key_frame flag to the
RtpEncodingParameters.

WebRTC CL:
https://webrtc-review.googlesource.com/c/src/+/286741
Spec PRs:
w3c/webrtc-pc#2885
w3c/webrtc-extensions#167

Chromestatus feature:
https://chromestatus.com/feature/5161082937409536

BUG=chromium:1354101

Change-Id: I5bfe266eac5990b1921212babdee1af35edc4242
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4643591
Reviewed-by: Harald Alvestrand <hta@chromium.org>
Commit-Queue: Philipp Hancke <phancke@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#1249577}

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.

adding a new RTCSetParameterOptions object which has a sequence of
RTCEncodingOptions similar to
WebCodecs VideoEncoderEncodeOptions
  https://w3c.github.io/webcodecs/#dictdef-videoencoderencodeoptions
and its keyFrame flag.

On the native side, this adds the request_key_frame flag to the
RtpEncodingParameters.

WebRTC CL:
  https://webrtc-review.googlesource.com/c/src/+/286741
Spec PRs:
  w3c/webrtc-pc#2885
  w3c/webrtc-extensions#167

Chromestatus feature:
  https://chromestatus.com/feature/5161082937409536

BUG=chromium:1354101

Change-Id: I5bfe266eac5990b1921212babdee1af35edc4242
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4643591
Reviewed-by: Harald Alvestrand <hta@chromium.org>
Commit-Queue: Philipp Hancke <phancke@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#1249577}
@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 3be83c4 into master Jan 19, 2024
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-4643591 branch January 19, 2024 23:17
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.

3 participants