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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-append-buffer.html

Issue 2707243006: [SharedArrayBuffer] Prevent SharedArrayBuffer being used in Web APIs (Closed)
Patch Set: update comment, add TODO Created 3 years, 8 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: third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-append-buffer.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-append-buffer.html b/third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-append-buffer.html
index ce6044796f26c2f5ac54c0e8e7d9c655fd95ff53..1d25402fafae9e85a3ef328770ca1e1f9eb12e68 100644
--- a/third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-append-buffer.html
+++ b/third_party/WebKit/LayoutTests/http/tests/media/media-source/mediasource-append-buffer.html
@@ -550,6 +550,22 @@
test.done();
}, 'Test appending null.');
+ if (window.SharedArrayBuffer) {
+ mediasource_test(function(test, mediaElement, mediaSource)
+ {
+ var sourceBuffer = mediaSource.addSourceBuffer(MediaSourceUtil.VIDEO_ONLY_TYPE);
+
+ test.expectEvent(sourceBuffer, 'updatestart', 'Append started.');
+ test.expectEvent(sourceBuffer, 'update', 'Append success.');
+ test.expectEvent(sourceBuffer, 'updateend', 'Append ended.');
+
+ assert_throws( { name: 'TypeError'} ,
+ function() { sourceBuffer.appendBuffer(new Uint8Array(new SharedArrayBuffer(16))); },
+ 'appendBuffer() of SharedArrayBuffer view throws an exception.');
+ test.done();
+ }, 'Test appending SharedArrayBuffer view.');
+ }
+
mediasource_testafterdataloaded(function(test, mediaElement, mediaSource, segmentInfo, sourceBuffer, mediaData)
{
mediaSource.removeSourceBuffer(sourceBuffer);

Powered by Google App Engine
This is Rietveld 408576698