Chromium Code Reviews Chromium Code Reviews
Issue 742653002:
MSE: Add layout test for 'decode' error via an algorithm (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| OLD | NEW |
|---|---|
| 1 <!DOCTYPE html> | 1 <!DOCTYPE html> |
| 2 <html> | 2 <html> |
| 3 <head> | 3 <head> |
| 4 <script src="/w3c/resources/testharness.js"></script> | 4 <script src="/w3c/resources/testharness.js"></script> |
| 5 <script src="/w3c/resources/testharnessreport.js"></script> | 5 <script src="/w3c/resources/testharnessreport.js"></script> |
| 6 <script src="mediasource-util.js"></script> | 6 <script src="mediasource-util.js"></script> |
| 7 <link rel='stylesheet' href='/w3c/resources/testharness.css'> | 7 <link rel='stylesheet' href='/w3c/resources/testharness.css'> |
| 8 </head> | 8 </head> |
| 9 <body> | 9 <body> |
| 10 <div id="log"></div> | 10 <div id="log"></div> |
| (...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 136 }); | 136 }); |
| 137 | 137 |
| 138 test.waitForExpectedEvents(function() | 138 test.waitForExpectedEvents(function() |
| 139 { | 139 { |
| 140 assert_true(mediaElement.error != null); | 140 assert_true(mediaElement.error != null); |
| 141 assert_equals(mediaElement.error.code, MediaError.MEDIA_ERR_ NETWORK); | 141 assert_equals(mediaElement.error.code, MediaError.MEDIA_ERR_ NETWORK); |
| 142 assert_equals(mediaSource.readyState, "ended"); | 142 assert_equals(mediaSource.readyState, "ended"); |
| 143 test.done(); | 143 test.done(); |
| 144 }); | 144 }); |
| 145 }, "Signaling 'network' error via endOfStream() after initialization segment has been appended and the HTMLMediaElement has reached HAVE_METADATA.") ; | 145 }, "Signaling 'network' error via endOfStream() after initialization segment has been appended and the HTMLMediaElement has reached HAVE_METADATA.") ; |
| 146 | |
| 147 ErrorTest(function(test, mediaElement, mediaSource, segmentInfo, sou rceBuffer, mediaData) | |
| 148 { | |
| 149 assert_equals(mediaElement.readyState, HTMLMediaElement.HAVE_NOT HING); | |
| 150 | |
| 151 var mediaSegment = MediaSourceUtil.extractSegmentData(mediaData, segmentInfo.media[0]); | |
| 152 var index = segmentInfo.init.size + (mediaSegment.length + 1) / 2; | |
|
philipj_slow
2014/11/20 09:02:02
I don't understand the +1 here. The lowest index o
jiajia.qin
2014/11/20 10:12:23
I just follow 'mediasource-append-buffer.html'. In
philipj_slow
2014/11/20 10:45:45
In that case, please use - 1 instead in both tests
| |
| 153 // Corrupt the media data from index of mediaData, so it can sig nal 'decode' error. | |
| 154 // Here use mediaSegment to replace the original mediaData[index , index + mediaSegment.length] | |
|
philipj_slow
2014/11/20 09:02:02
If I'm reading this correctly, we'll the init segm
jiajia.qin
2014/11/20 10:12:24
Yes, it is :)
| |
| 155 mediaData.set(mediaSegment, index); | |
| 156 | |
| 157 test.expectEvent(mediaElement, "error", "mediaElement error."); | |
| 158 sourceBuffer.appendBuffer(mediaData); | |
| 159 | |
| 160 test.waitForExpectedEvents(function() | |
| 161 { | |
| 162 assert_equals(mediaElement.readyState, HTMLMediaElement.HAVE _METADATA); | |
| 163 assert_true(mediaElement.error != null); | |
| 164 assert_equals(mediaElement.error.code, MediaError.MEDIA_ERR_ DECODE); | |
| 165 test.done(); | |
| 166 }); | |
| 167 }, "Signaling 'decode' error via segment parser loop algorithm after initialization segment and partial media segment has been appended."); | |
| 146 </script> | 168 </script> |
| 147 </body> | 169 </body> |
| 148 </html> | 170 </html> |
| OLD | NEW |
