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

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

Issue 212953004: Have SourceBuffer throw TypeError for non-finite doubles. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 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
« no previous file with comments | « no previous file | LayoutTests/http/tests/media/media-source/mediasource-timestamp-offset.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/http/tests/media/media-source/mediasource-appendwindow.html
diff --git a/LayoutTests/http/tests/media/media-source/mediasource-appendwindow.html b/LayoutTests/http/tests/media/media-source/mediasource-appendwindow.html
index 289fb7c93297ba1e1c7e717e61c5f9223a5fe476..60f1bb7911318143f4d98c5503ef6f58c809f8d7 100644
--- a/LayoutTests/http/tests/media/media-source/mediasource-appendwindow.html
+++ b/LayoutTests/http/tests/media/media-source/mediasource-appendwindow.html
@@ -32,15 +32,15 @@
assert_true(sourceBuffer != null, "New SourceBuffer returned");
sourceBuffer.appendWindowEnd = 500.0;
- assert_throws("TypeMismatchError",
+ assert_throws({name: "TypeError"},
acolwell GONE FROM CHROMIUM 2014/03/26 16:41:36 nit: Why can't we just use "TypeError"? It seems o
sof 2014/03/26 16:43:37 It'd be nice, but testharness.js only handles DOM
function() { sourceBuffer.appendWindowStart = Number.NEGATIVE_INFINITY; },
"set appendWindowStart throws an exception for Number.NEGATIVE_INFINITY.");
- assert_throws("TypeMismatchError",
+ assert_throws({name: "TypeError"},
function() { sourceBuffer.appendWindowStart = Number.POSITIVE_INFINITY; },
"set appendWindowStart throws an exception for Number.POSITIVE_INFINITY.");
- assert_throws("TypeMismatchError",
+ assert_throws({name: "TypeError"},
function() { sourceBuffer.appendWindowStart = Number.NaN; },
"set appendWindowStart throws an exception for Number.NaN.");
@@ -101,7 +101,7 @@
sourceBuffer.abort();
assert_equals(sourceBuffer.appendWindowStart, 0, "appendWindowStart is 0 after an abort'");
- assert_equals(sourceBuffer.appendWindowEnd, Number.POSITIVE_INFINITY,
+ assert_equals(sourceBuffer.appendWindowEnd, Number.POSITIVE_INFINITY,
"appendWindowStart is POSITIVE_INFINITY after an abort");
test.waitForExpectedEvents(function()
{
« no previous file with comments | « no previous file | LayoutTests/http/tests/media/media-source/mediasource-timestamp-offset.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698