Skip to content

Commit 1465875

Browse files
DrekabiSkyBlack1225
authored andcommitted
Removed larger test values to improve test performance.
Differential Revision: https://phabricator.services.mozilla.com/D178875 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1834531 gecko-commit: 93d1b1ef74ee305d8ba91db42b16a500e248e34a gecko-reviewers: jib
1 parent 1580d4a commit 1465875

File tree

1 file changed

+5
-11
lines changed

1 file changed

+5
-11
lines changed

webrtc-extensions/RTCRtpReceiver-jitterBufferTarget-stats.html

+5-11
Original file line numberDiff line numberDiff line change
@@ -21,18 +21,10 @@
2121
"measure raising video jitterBufferTarget to 250");
2222
async_promise_test(t => applyJitterBufferTarget(t, "audio", 250),
2323
"measure raising audio jitterBufferTarget to 250");
24-
async_promise_test(t => applyJitterBufferTarget(t, "video", 500),
25-
"measure raising video jitterBufferTarget to 500");
26-
async_promise_test(t => applyJitterBufferTarget(t, "audio", 500),
27-
"measure raising audio jitterBufferTarget to 500");
2824
async_promise_test(t => applyJitterBufferTarget(t, "video", 250, 150),
2925
"measure lowering video jitterBufferTarget to 150");
3026
async_promise_test(t => applyJitterBufferTarget(t, "audio", 250, 150),
3127
"measure lowering audio jitterBufferTarget to 150");
32-
async_promise_test(t => applyJitterBufferTarget(t, "video", 400, 250),
33-
"measure lowering video jitterBufferTarget to 300");
34-
async_promise_test(t => applyJitterBufferTarget(t, "audio", 400, 250),
35-
"measure lowering audio jitterBufferTarget to 300");
3628

3729
async function applyJitterBufferTarget(t, kind, target, targetToLower) {
3830
const caller = new RTCPeerConnection();
@@ -67,7 +59,8 @@
6759
async function measureDelayFromStats(t, receiver, callee, target, kind) {
6860
const lowerBound = target * 0.7;
6961
const upperBound = target * 1.3;
70-
let delay, oldInboundStats, prevDelay, rateOfChange;
62+
let delay, oldInboundStats, prevDelay, avgChange;
63+
let rateOfChange = 0;
7164
let numDelayMeasurements = 1;
7265

7366
for (let statChecks = 0; statChecks < 40; statChecks++) {
@@ -79,7 +72,8 @@
7972
delay = ((inboundStats.jitterBufferDelay - oldInboundStats.jitterBufferDelay) /
8073
(inboundStats.jitterBufferEmittedCount - oldInboundStats.jitterBufferEmittedCount) * 1000);
8174
if (prevDelay) {
82-
rateOfChange = (delay - prevDelay) / numDelayMeasurements;
75+
rateOfChange += (delay - prevDelay);
76+
avgChange = rateOfChange / numDelayMeasurements;
8377
numDelayMeasurements++;
8478
}
8579
prevDelay = delay;
@@ -90,7 +84,7 @@
9084
oldInboundStats = inboundStats;
9185
}
9286
t.step(() => assert_between_inclusive(delay , lowerBound, upperBound,
93-
`${kind} delay is within bounds rateOfChange ${rateOfChange} over ${numDelayMeasurements} measurements`));
87+
`${kind} delay is within bounds avgChange ${avgChange} over ${numDelayMeasurements} measurements`));
9488
return delay;
9589
}
9690
</script>

0 commit comments

Comments
 (0)