Setting "" as <track> 'src' should fail the first time

When setting a track .src to "", we would resolve the URL to a null URL, which matched the default initialized HTMLTrackElement::url_, meaning that readyState was not advanced and no 'error' event fired in this case. Extend the "url == url_" check with a check for readyState differing from 'none'. Also rewrite HTMLTrackElement::getReadyState to avoid using EnsureTrack(). Bug: 811713 Change-Id: Icd190492639c53e09a7e018202b69ac6e919a94c Reviewed-on: https://chromium-review.googlesource.com/916004 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by: srirama chandra sekhar <srirama.m@samsung.com> Cr-Commit-Position: refs/heads/master@{#536685}
web-platform-tests · Feb 14, 2018 · ca76cf8 · ca76cf8
1 parent 783faf5
commit ca76cf8
Showing 1 changed file with 21 additions and 0 deletions.
diff --git a/html/semantics/embedded-content/media-elements/track/track-element/src-empty-string.html b/html/semantics/embedded-content/media-elements/track/track-element/src-empty-string.html
@@ -0,0 +1,21 @@
+<!DOCTYPE html>
+<title>Setting HTMLTrackElement.src to the empty string fires 'error' and sets readyState to ERROR</title>
+<link rel="help" href="https://html.spec.whatwg.org/multipage/media.html#sourcing-out-of-band-text-tracks">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<video></video>
+<script>
+async_test(t => {
+  let track = document.createElement("track");
+  track.src = '';
+  track.default = true;
+  track.onerror = t.step_func_done(() => {
+    assert_equals(track.readyState, HTMLTrackElement.ERROR);
+  });
+  track.onload = t.unreached_func('fired load');
+
+  assert_equals(track.readyState, HTMLTrackElement.NONE);
+
+  document.querySelector('video').appendChild(track);
+});
+</script>