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

Options to guide codec selection #326

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@martinthomson
Member

martinthomson commented Oct 12, 2015

This is pretty primitive, but I think that it is sufficient.

<p>The order of this sequence determines the preference order.
Codecs that appear first in this list MUST be selected by the user
agent in preference to those that appear after.</p>
</dd>

This comment has been minimized.

@alvestrand

alvestrand Oct 15, 2015

Contributor

How does this work if this option is not specified?
I think the text should say that "if left off, browser can negotiate everything".

<dl class="idl" title="dictionary RTCRtpCodecCapability">
<dt>DOMString mimeType</dt>
<dl class="idl" title="interface RTCRtpCodecCapability">
<dt>readonly attribute DOMString mimeType</dt>

This comment has been minimized.

@alvestrand

alvestrand Oct 15, 2015

Contributor

Please explain the reasons for the dictionary -> interface change here.

This comment has been minimized.

@martinthomson

martinthomson Oct 15, 2015

Member

If this is not an interface, the application can construct preferred codecs that the browser didn't agree to supporting.

@martinthomson

This comment has been minimized.

Member

martinthomson commented Oct 21, 2015

I'm going to drop this change, and await the merging of #293 before generating a new PR.

@martinthomson martinthomson deleted the martinthomson:prenup branch Oct 21, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment