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

Unified Diff: LayoutTests/http/tests/media/media-source/mediasource-util.js

Issue 273153002: MSE: Add basic sequence AppendMode layout tests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix nits Created 6 years, 7 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
« no previous file with comments | « LayoutTests/http/tests/media/media-source/mediasource-sequencemode-append-buffer-expected.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/http/tests/media/media-source/mediasource-util.js
diff --git a/LayoutTests/http/tests/media/media-source/mediasource-util.js b/LayoutTests/http/tests/media/media-source/mediasource-util.js
index 7bf4d461ee551306564a73b2a95fcc43bfcdb45b..5f0c4075d030df88ad54f15debea4717dbd42dd0 100644
--- a/LayoutTests/http/tests/media/media-source/mediasource-util.js
+++ b/LayoutTests/http/tests/media/media-source/mediasource-util.js
@@ -17,22 +17,22 @@
bufferedRangeEndBeforeEndOfStream: 6.040,
init: { offset: 0, size: 4357 },
media: [
- { offset: 4357, size: 11830, timecode: 0 },
- { offset: 16187, size: 12588, timecode: 0.385 },
- { offset: 28775, size: 14588, timecode: 0.779 },
- { offset: 43363, size: 13023, timecode: 1.174 },
- { offset: 56386, size: 13127, timecode: 1.592 },
- { offset: 69513, size: 14456, timecode: 1.987 },
- { offset: 83969, size: 13458, timecode: 2.381 },
- { offset: 97427, size: 14566, timecode: 2.776 },
- { offset: 111993, size: 13201, timecode: 3.171 },
- { offset: 125194, size: 14061, timecode: 3.566 },
- { offset: 139255, size: 15353, timecode: 3.96 },
- { offset: 154608, size: 13618, timecode: 4.378 },
- { offset: 168226, size: 15094, timecode: 4.773 },
- { offset: 183320, size: 13069, timecode: 5.168 },
- { offset: 196389, size: 13788, timecode: 5.563 },
- { offset: 210177, size: 9009, timecode: 5.957 },
+ { offset: 4357, size: 11830, timecode: 0, highest_end_time: 0.398 },
+ { offset: 16187, size: 12588, timecode: 0.385, highest_end_time: 0.797 },
+ { offset: 28775, size: 14588, timecode: 0.779, highest_end_time: 1.195 },
+ { offset: 43363, size: 13023, timecode: 1.174, highest_end_time: 1.593 },
+ { offset: 56386, size: 13127, timecode: 1.592, highest_end_time: 1.992 },
+ { offset: 69513, size: 14456, timecode: 1.987, highest_end_time: 2.39 },
+ { offset: 83969, size: 13458, timecode: 2.381, highest_end_time: 2.789 },
+ { offset: 97427, size: 14566, timecode: 2.776, highest_end_time: 3.187 },
+ { offset: 111993, size: 13201, timecode: 3.171, highest_end_time: 3.585 },
+ { offset: 125194, size: 14061, timecode: 3.566, highest_end_time: 3.984 },
+ { offset: 139255, size: 15353, timecode: 3.96, highest_end_time: 4.382 },
+ { offset: 154608, size: 13618, timecode: 4.378, highest_end_time: 4.781 },
+ { offset: 168226, size: 15094, timecode: 4.773, highest_end_time: 5.179 },
+ { offset: 183320, size: 13069, timecode: 5.168, highest_end_time: 5.577 },
+ { offset: 196389, size: 13788, timecode: 5.563, highest_end_time: 5.976 },
+ { offset: 210177, size: 9009, timecode: 5.957, highest_end_time: 6.042 },
],
},
{
@@ -43,15 +43,19 @@
bufferedRangeEndBeforeEndOfStream: 6.0368,
init: { offset: 0, size: 1178 },
media: [
- { offset: 1246, size: 23828, timecode: 0 },
- { offset: 25142, size: 25394, timecode: 0.797 },
- { offset: 50604, size: 24761, timecode: 1.594 },
- { offset: 75433, size: 25138, timecode: 2.390 },
- { offset: 100639, size: 22935, timecode: 3.187 },
- { offset: 123642, size: 24995, timecode: 3.984},
- { offset: 148637, size: 24968, timecode: 4.781 },
- { offset: 173689, size: 19068, timecode: 5.578 },
- { offset: 192757, size: 200, timecode: 5.619 },
+ // FIXME: Fix these timecodes to be PTS, not DTS, and highest_end_times to correspond
+ // to highest PTS+duration, not highest DTS+duration. See http://crbug.com/373039.
+ // FIXME: Some segments are parsed to start with keyframe but DTS < PTS. See
+ // http://crbug.com/371947 and http://crbug.com/367786.
+ { offset: 1246, size: 23828, timecode: 0, highest_end_time: 0.835917 },
+ { offset: 25142, size: 25394, timecode: 0.797, highest_end_time: 1.625395 },
+ { offset: 50604, size: 24761, timecode: 1.594, highest_end_time: 2.414874 },
+ { offset: 75433, size: 25138, timecode: 2.390, highest_end_time: 3.227572 },
+ { offset: 100639, size: 22935, timecode: 3.187, highest_end_time: 4.017051 },
+ { offset: 123642, size: 24995, timecode: 3.984, highest_end_time: 4.806529 },
+ { offset: 148637, size: 24968, timecode: 4.781, highest_end_time: 5.619228 },
+ { offset: 173689, size: 19068, timecode: 5.578, highest_end_time: 6.0424 },
+ { offset: 192757, size: 200, timecode: 5.619, highest_end_time: 6.0424 },
],
}
];
« no previous file with comments | « LayoutTests/http/tests/media/media-source/mediasource-sequencemode-append-buffer-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698