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

Add hardwareAccelerated field to RTCRtpCodecCapability #2355

Closed
drkron opened this issue Nov 8, 2019 · 5 comments
Closed

Add hardwareAccelerated field to RTCRtpCodecCapability #2355

drkron opened this issue Nov 8, 2019 · 5 comments

Comments

@drkron
Copy link
Contributor

drkron commented Nov 8, 2019

It is sometimes beneficial to be able to tell beforehand if a particular codec has support for hardware acceleration. Currently, the only way to determine if there is hardware acceleration is to start a stream that is using the particular codec and look at the implementation name in getStats().

By knowing beforehand if there is hardware support, a client could use this information to signal back to the server that it should avoid/prefer a particular codec. This is of special importance for high bitrate streams where it's likely that a client will not be able to decode the stream in full fps if there is no support for hardware acceleration.

@aboba
Copy link
Contributor

aboba commented Nov 8, 2019

This is likely to be flagged in a privacy review. See: w3c/webrtc-svc#22

@alvestrand
Copy link
Contributor

We do have a feature freeze for WebRTC 1.0 - the new field wouldn't be implemented when added, and is therefore automatically a Feature At Risk.
Can we add it to the "misc extensions" spec instead?

@drkron
Copy link
Contributor Author

drkron commented Nov 11, 2019

I'm aware of the feature freeze, but was advised to float this here initially.
Where do I find the "misc extensions" spec?

@alvestrand
Copy link
Contributor

@henbos
Copy link
Contributor

henbos commented Nov 25, 2019

Moved this to https://github.com/henbos/webrtc-extensions/issues/21. Closing.

@henbos henbos closed this as completed Nov 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants