Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(166)

Unified Diff: LayoutTests/http/tests/media/media-source/mediasource-remove.html

Issue 215863002: MSE: Update tests to expect failure w.r.t. Chromium LegacyFrameProcessor duration logic change (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Update unprefixed MP4 layout test duration Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/http/tests/media/media-source/mediasource-remove.html
diff --git a/LayoutTests/http/tests/media/media-source/mediasource-remove.html b/LayoutTests/http/tests/media/media-source/mediasource-remove.html
index 6cf84a971a22c38bc98eb818729feae07c657f92..a65e90962512472434995475ea17f9f17596622c 100644
--- a/LayoutTests/http/tests/media/media-source/mediasource-remove.html
+++ b/LayoutTests/http/tests/media/media-source/mediasource-remove.html
@@ -156,11 +156,11 @@
test.waitForExpectedEvents(function()
{
- var duration = segmentInfo.duration.toFixed(3);
+ var bufferedRangeEnd = segmentInfo.bufferedRangeEndBeforeEndOfStream.toFixed(3);
var subType = MediaSourceUtil.getSubType(segmentInfo.type);
- assertBufferedEquals(sourceBuffer, "{ [0.000, " + duration + ") }", "Initial buffered range.");
- callback(test, sourceBuffer, duration, subType);
+ assertBufferedEquals(sourceBuffer, "{ [0.000, " + bufferedRangeEnd + ") }", "Initial buffered range.");
+ callback(test, sourceBuffer, bufferedRangeEnd, subType);
});
}, description);
};
@@ -179,16 +179,16 @@
});
}
- removeAppendedDataTests(function(test, sourceBuffer, duration, subType)
+ removeAppendedDataTests(function(test, sourceBuffer, bufferedRangeEnd, subType)
{
removeAndCheckBufferedRanges(test, sourceBuffer, 0, Number.POSITIVE_INFINITY, "{ }");
}, "Test removing all appended data.");
- removeAppendedDataTests(function(test, sourceBuffer, duration, subType)
+ removeAppendedDataTests(function(test, sourceBuffer, bufferedRangeEnd, subType)
{
var expectations = {
- webm: ("{ [3.187, " + duration + ") }"),
- mp4: ("{ [3.021, " + duration + ") }"),
+ webm: ("{ [3.187, " + bufferedRangeEnd + ") }"),
+ mp4: ("{ [3.021, " + bufferedRangeEnd + ") }"),
};
// Note: Range doesn't start exactly at the end of the remove range because there isn't
@@ -196,11 +196,11 @@
removeAndCheckBufferedRanges(test, sourceBuffer, 0, 3, expectations[subType]);
}, "Test removing beginning of appended data.");
- removeAppendedDataTests(function(test, sourceBuffer, duration, subType)
+ removeAppendedDataTests(function(test, sourceBuffer, bufferedRangeEnd, subType)
{
var expectations = {
- webm: ("{ [0.000, 1.012) [3.187, " + duration + ") }"),
- mp4: ("{ [0.000, 1.022) [3.021, " + duration + ") }"),
+ webm: ("{ [0.000, 1.012) [3.187, " + bufferedRangeEnd + ") }"),
+ mp4: ("{ [0.000, 1.022) [3.021, " + bufferedRangeEnd + ") }"),
};
// Note: The first resulting range ends slightly after start because the removal algorithm only removes
@@ -209,7 +209,7 @@
removeAndCheckBufferedRanges(test, sourceBuffer, 1, 3, expectations[subType]);
}, "Test removing the middle of appended data.");
- removeAppendedDataTests(function(test, sourceBuffer, duration, subType)
+ removeAppendedDataTests(function(test, sourceBuffer, bufferedRangeEnd, subType)
{
var expectations = {
webm: "{ [0.000, 1.012) }",

Powered by Google App Engine
This is Rietveld 408576698