Re: [webrtc-rtptransport] How do the two `sendRtp` functions interact? (#32)

I thought they would be merged as a single method, ie `Promise<RtpSendResult> sendRtp(RtpPacketInit packet, optional RtpSendOptions options = {});`
with usage:
```
// Send packet with the built in pacer:
vanillaSendStream.sendRtp(packet);

// Send pacedPacket with specified pacing:
pacedSendStream.sendRtp(pacedPacket, {sendTime: 3});

// Actually listen to the send result (probably actually replaced with a batch interface per issue #20)
bweSendStream.sendRtp(pacedPacket, {sendTime: congestionController.pace(pacedPacket)}).then((s) => congestionController.sent(s));
```

If you're concerned about preventing JS from trying to send both paced and unpaced packets on the same send stream, we'll have the same issue with separate functions.

-- 
GitHub Notification of comment by tonyherre
Please view or discuss this issue at https://github.com/w3c/webrtc-rtptransport/issues/32#issuecomment-2144849083 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Monday, 3 June 2024 10:30:35 UTC