Bug 1082236 - relax RTCP timestamp verification in case of steeplechase. r=spolk

This commit is contained in:
Nils Ohlmeier [:drno] 2014-10-13 17:00:00 +02:00
parent 169f4c3693
commit ec1f0fee2b
2 changed files with 9 additions and 3 deletions

View File

@ -2439,7 +2439,7 @@ PeerConnectionWrapper.prototype = {
* @param {object} stats
* The stats to check from this PeerConnectionWrapper
*/
checkStats : function PCW_checkStats(stats) {
checkStats : function PCW_checkStats(stats, twoMachines) {
function toNum(obj) {
return obj? obj : 0;
}
@ -2461,7 +2461,13 @@ PeerConnectionWrapper.prototype = {
// validate stats
ok(res.id == key, "Coherent stats id");
var nowish = Date.now() + 1000; // TODO: clock drift observed
if (twoMachines) {
nowish += 10000; // let's be very relaxed about clock sync
}
var minimum = this.whenCreated - 1000; // on Windows XP (Bug 979649)
if (twoMachines) {
minimum -= 10000; // let's be very relaxed about clock sync
}
if (isWinXP) {
todo(false, "Can't reliably test rtcp timestamps on WinXP (Bug 979649)");
} else {

View File

@ -478,7 +478,7 @@ var commandsPeerConnection = [
'PC_LOCAL_CHECK_STATS',
function (test) {
test.pcLocal.getStats(null, function(stats) {
test.pcLocal.checkStats(stats);
test.pcLocal.checkStats(stats, test.steeplechase);
test.next();
});
}
@ -487,7 +487,7 @@ var commandsPeerConnection = [
'PC_REMOTE_CHECK_STATS',
function (test) {
test.pcRemote.getStats(null, function(stats) {
test.pcRemote.checkStats(stats);
test.pcRemote.checkStats(stats, test.steeplechase);
test.next();
});
}