Bug 1529821 [wpt PR 15506] - Surface dtlsTransport via state-surfacer, a=testonly
authorHarald Alvestrand <hta@chromium.org>
Wed, 06 Mar 2019 12:34:26 +0000
changeset 522547 1966a78128ba364656ebe9aee80f7f5a1b63e90b
parent 522546 c2de5b0649719b063e0c85d084c37b99db7cc11b
child 522548 8b69d51eaae559880834e12b2dceba7d9f055b79
push id10871
push usercbrindusan@mozilla.com
push dateMon, 18 Mar 2019 15:49:32 +0000
treeherdermozilla-beta@018abdd16060 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstestonly
bugs1529821, 15506, 907849, 1455896, 633958
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1529821 [wpt PR 15506] - Surface dtlsTransport via state-surfacer, a=testonly Automatic update from web-platform-tests Surface dtlsTransport via state-surfacer This allows for more consistent state information. Bug: chromium:907849 Change-Id: I8b71b3ec3eb1acf83624969190519c5f18b2a19e Reviewed-on: https://chromium-review.googlesource.com/c/1455896 Commit-Queue: Harald Alvestrand <hta@chromium.org> Reviewed-by: Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#633958} -- wpt-commits: 36baff5e6981e5a235ed1c5f13b45ffed241770b wpt-pr: 15506
testing/web-platform/tests/webrtc/RTCRtpSender-transport.https.html
--- a/testing/web-platform/tests/webrtc/RTCRtpSender-transport.https.html
+++ b/testing/web-platform/tests/webrtc/RTCRtpSender-transport.https.html
@@ -64,20 +64,24 @@
         assert_equals(caller_transceiver1.sender.transport,
                       caller_transceiver2.sender.transport);
       } else {
         assert_not_equals(caller_transceiver1.sender.transport,
                           caller_transceiver2.sender.transport);
       }
       await callee.setRemoteDescription(offer);
       const [callee_transceiver1, callee_transceiver2] = callee.getTransceivers();
+      // According to spec, setRemoteDescription only updates the transports
+      // if the remote description is an answer.
+      assert_equals(callee_transceiver1.receiver.transport, null);
+      assert_equals(callee_transceiver2.receiver.transport, null);
+      const answer = await callee.createAnswer();
+      await callee.setLocalDescription(answer);
       assert_not_equals(callee_transceiver1.receiver.transport, null);
       assert_not_equals(callee_transceiver2.receiver.transport, null);
-      const answer = await callee.createAnswer();
-      await callee.setLocalDescription(answer);
       // At this point, bundle should have kicked in.
       assert_equals(callee_transceiver1.receiver.transport,
                     callee_transceiver2.receiver.transport);
       await caller.setRemoteDescription(answer);
       assert_equals(caller_transceiver1.receiver.transport,
                     caller_transceiver2.receiver.transport);
     }, 'RTCRtpSender/receiver.transport at the right time, with bundle policy ' + bundle_policy);
   }