I see a TODO for "Handle trickle ICE candidates" here.
But there are already two methods, one that supports handling trickle ICE candidates and the other for setting the initial candidates: bool SetRemoteIceCandidate and bool SetRemoteIceCandidates.
Is this TODO about unifying the functionality into a single API or is it just a left over comment? If it's the latter case, do you think we have more of these left over comments lying around that needs clean up?
I see a TODO for "Handle trickle ICE candidates" here.
But there are already two methods, one that supports handling trickle ICE candidates and the other for setting the initial candidates:
bool SetRemoteIceCandidateandbool SetRemoteIceCandidates.Is this TODO about unifying the functionality into a single API or is it just a left over comment? If it's the latter case, do you think we have more of these left over comments lying around that needs clean up?