Commit cec942d9 authored by Chris Sharp's avatar Chris Sharp Committed by Commit Bot

Revert "[RTCInsertableStreams] Add new tests"

This reverts commit 3ab3a892.

Reason for revert: consistently fails on Linux Tests (dbg)(1)

[3865/4436] fast/peerconnection/RTCPeerConnection-insertable-streams.html failed unexpectedly (asserts failed)

Original change's description:
> [RTCInsertableStreams] Add new tests
> 
> These tests verify that data flows normally through senders/receivers
> of one kind when the peer connection has enabled insertable streams
> for the other kind.
> 
> These tests cover the changes in
> https://chromium-review.googlesource.com/c/chromium/src/+/2139231
> 
> Bug: 1068125
> Change-Id: If00ab2cc554eae8632cb0a4fb1d010a7fe5fba3f
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2139213
> Reviewed-by: Harald Alvestrand <hta@chromium.org>
> Commit-Queue: Guido Urdaneta <guidou@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#757060}

TBR=hta@chromium.org,guidou@chromium.org

Change-Id: I70968239d8112c53bd32058acfafacf6360ec2b8
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1068125
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2140037Reviewed-by: default avatarChris Sharp <csharp@chromium.org>
Commit-Queue: Chris Sharp <csharp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#757146}
parent 49392e1b
......@@ -231,60 +231,6 @@ promise_test(async t => {
return onmessagePromise;
}, 'RTCRtpSender readable stream transferred to a Worker and the Worker sends an RTCEncodedVideoFrame back');
async function testNormalDataFlowWithInsertableStreamsForOtherKind(t, normalKind) {
const flowAudio = (normalKind == 'audio');
const pcConfig = flowAudio
? {forceEncodedVideoInsertableStreams:true}
: {forceEncodedAudioInsertableStreams:true};
const caller = new RTCPeerConnection(pcConfig);
t.add_cleanup(() => caller.close());
const callee = new RTCPeerConnection(pcConfig);
t.add_cleanup(() => callee.close());
const stream = await navigator.mediaDevices.getUserMedia(
{audio: flowAudio, video: !flowAudio});
const track = stream.getTracks()[0];
t.add_cleanup(() => track.stop());
const sender = caller.addTrack(track, stream);
const ontrackPromise = new Promise((resolve,reject) => {
callee.ontrack = t.step_func(async e => {
let numGetStats = 0;
const intervalId = window.setInterval(async ()=> {
let statsReport = await callee.getStats();
statsReport.forEach(report => {
if (report.type == 'inbound-rtp' &&
report.kind == normalKind &&
report.bytesReceived > 0) {
clearInterval(intervalId);
resolve();
return;
}
});
if (++numGetStats >= 3) {
clearInterval(intervalId);
reject('No bytes received.');
return;
}
}, 50);
});
});
exchangeIceCandidates(caller, callee);
await doSignalingHandshake(caller, callee);
return ontrackPromise;
}
promise_test(t => {
return testNormalDataFlowWithInsertableStreamsForOtherKind(t, 'audio');
}, 'Audio flows when insertable streams is enabled for video and disabled for audio.');
promise_test(t => {
return testNormalDataFlowWithInsertableStreamsForOtherKind(t, 'video');
}, 'Video flows when insertable streams is enabled for audio and disabled for audio.');
</script>
</body>
</html>
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment