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

Side by Side Diff: LayoutTests/http/tests/media/media-source/mediasource-sourcebufferlist.html

Issue 761773002: MSE: Remove "item" indexed property getter from SourceBufferList (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years 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 unified diff | Download patch
OLDNEW
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 7
8 <link rel='stylesheet' href='/w3c/resources/testharness.css'> 8 <link rel='stylesheet' href='/w3c/resources/testharness.css'>
9 </head> 9 </head>
10 <body> 10 <body>
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 105
106 test.waitForExpectedEvents(function() 106 test.waitForExpectedEvents(function()
107 { 107 {
108 assert_equals(mediaElement.error.code, MediaError.MEDIA_ERR_SR C_NOT_SUPPORTED, "Expected error code"); 108 assert_equals(mediaElement.error.code, MediaError.MEDIA_ERR_SR C_NOT_SUPPORTED, "Expected error code");
109 assert_equals(mediaSource.readyState, "closed", "mediaSource i s closed."); 109 assert_equals(mediaSource.readyState, "closed", "mediaSource i s closed.");
110 assert_equals(mediaSource.sourceBuffers.length, 0, "mediaSourc e sourceBuffers"); 110 assert_equals(mediaSource.sourceBuffers.length, 0, "mediaSourc e sourceBuffers");
111 assert_equals(mediaSource.activeSourceBuffers.length, 0, "medi aSource activeSourceBuffers"); 111 assert_equals(mediaSource.activeSourceBuffers.length, 0, "medi aSource activeSourceBuffers");
112 test.done(); 112 test.done();
113 }); 113 });
114 }, "Test removesourcebuffer event firing on sourceBuffers and activeSo urceBuffers when MediaSource closes.", { allow_media_element_error: true } ); 114 }, "Test removesourcebuffer event firing on sourceBuffers and activeSo urceBuffers when MediaSource closes.", { allow_media_element_error: true } );
115
116 mediasource_testafterdataloaded(function(test, mediaElement, mediaSour ce, segmentInfo, sourceBuffer, mediaData)
117 {
118 // Basic checks
119 assert_equals(mediaSource.sourceBuffers.length, 1, "mediaSource so urceBuffers length");
120 assert_equals(mediaSource.sourceBuffers[0], sourceBuffer, "matchin g sourceBuffer");
121 assert_not_equals(undefined, null, "null is different from undefin ed");
philipj_slow 2014/11/26 18:52:34 A bit paranoid, but OK :)
122
123 // SourceBufferList's indexed property getter is anonymous. At lea st ensure no 'item' is defined.
124 assert_equals(mediaSource.sourceBuffers.item, undefined, "'item' i s undefined for MediaSource.sourceBuffers");
125
126 // Out-of-range indices should result in undefined.
127 assert_equals(mediaSource.sourceBuffers[1], undefined, "undefined result for out-of-range (+1 index)");
128 assert_equals(mediaSource.sourceBuffers[2], undefined, "undefined result for out-of-range (+2 index)");
129 assert_equals(mediaSource.sourceBuffers[undefined], undefined, "un defined result for out-of-range (undefined index)")
130 assert_equals(mediaSource.sourceBuffers[null], undefined, "undefin ed result for out-of-range (null index)");
131 assert_equals(mediaSource.sourceBuffers[""], undefined, "undefined result for out-of-range (empty string index)");
132 assert_equals(mediaSource.sourceBuffers["invalid index"], undefine d, "undefined result for out-of-range (string index)");
133
134 test.done();
135 }, "Test out-of-range SourceBufferList indexed property getter behavio r");
115 </script> 136 </script>
116 </body> 137 </body>
117 </html> 138 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698