webrtc wpt: reject SDES offer with InvalidAccessError
as agreed upon in
https://github.com/w3c/webrtc-pc/issues/2672
BUG=None
Change-Id: Ie3d97395220d0d1694d79a3cc9e28205d8082610
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5200862
Commit-Queue: Philipp Hancke <phancke@microsoft.com>
Reviewed-by: Harald Alvestrand <hta@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1248055}
diff --git a/webrtc/protocol/sdes-dont-dont-dont.html b/webrtc/protocol/sdes-dont-dont-dont.html
index e938c84..dcf7ad1 100644
--- a/webrtc/protocol/sdes-dont-dont-dont.html
+++ b/webrtc/protocol/sdes-dont-dont-dont.html
@@ -37,19 +37,13 @@
pc.addTransceiver('audio');
const offer = await pc.createOffer();
assert_false(offer.sdp.includes('\na=crypto:'));
-}, 'does not create offers with SDES');
+}, 'Does not create offers with SDES');
-promise_test(async t => {
+promise_test(t => {
const pc = new RTCPeerConnection();
t.add_cleanup(() => pc.close());
- try {
- await pc.setRemoteDescription({type: 'offer', sdp});
- assert_unreached("Must not accept SDP without fingerprint");
- } catch (e) {
- // TODO: which error is correct? See
- // https://github.com/w3c/webrtc-pc/issues/2672
- assert_true(['OperationError', 'InvalidAccessError'].includes(e.name));
- }
+ return promise_rejects_dom(t, 'InvalidAccessError',
+ pc.setRemoteDescription({type: 'offer', sdp}));
}, 'rejects a remote offer that only includes SDES and no DTLS fingerprint');
</script>