-
Notifications
You must be signed in to change notification settings - Fork 115
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
Adding canTrickleIceCandidates #20
Conversation
See comment in JSEP pull, should this be .canTrickleIceCandidates to better match addIceCandidate() and onicecandidate |
You just want to punish me for doing the work. I'll fix it up eventually. |
not sure I would call adding 10 more characters on line 1245 a punishment, but if you want to two-step it, sure. |
Attribute it to snark. I've a large enough pile of edits to do. This will hardly make a dent. |
I don't understand. This attribute Isn't the application supposed to decide how to signal everything? |
LGTM |
LGTM |
1 similar comment
LGTM |
Still LGTM |
Any Editor willing to merge? |
As discussed in TPAC '14