Adjust track vs. video loading in track-remove-insert-ready-state.html

When the test was upstreamed to WPT in
d86c4f8fdaaf4da023fc71172e20508f2ffd31ec, the <script> element that
starts the load for the video resource was moved to be a child of the
<video> element. This means that the load of the <track> element will
start when the <video> elements end tag is encountered, while the load
of the video resource starts a bit earlier. Thus if the video resource
some completes loading (enough) before the <video> element is closed by
the parser, the <track> element may not have been selected and set to
'showing' yet. This can lead to flakiness.

Restore the location of the <script> that defines the async_test() to
that which it had before the export.

Bug: 1453552
Change-Id: I02efc27e36ca288754d0854aee14637415baea6b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4624051
Reviewed-by: Dale Curtis <[email protected]>
Commit-Queue: Fredrik Söderquist <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1160554}

Author: Fredrik Söderquist

html/semantics/embedded-content/media-elements/track/track-element/track-remove-insert-ready-state.html

@@ -5,34 +5,34 @@
 <script src="/resources/testharnessreport.js"></script>
 <video>
     <track src="resources/no-webvtt.vtt" kind="captions" default>
-    <script>
-    async_test(function(t) {
-        var video = document.querySelector('video');
-        video.src = getVideoURI('/media/test');
-        video.oncanplaythrough = t.step_func(canplaythrough);
+</video>
+<script>
+  async_test(function(t) {
+    var video = document.querySelector('video');
+    video.src = getVideoURI('/media/test');
+    video.oncanplaythrough = t.step_func(canplaythrough);
 
-        function canplaythrough() {
-            video.oncanplaythrough = null;
-            var track = document.querySelector('track');
+    function canplaythrough() {
+      video.oncanplaythrough = null;
+      var track = document.querySelector('track');
 
-            // Track should have error as ready state.
-            assert_equals(track.readyState, HTMLTrackElement.ERROR);
+      // Track should have error as ready state.
+      assert_equals(track.readyState, HTMLTrackElement.ERROR);
 
-            // Remove the video element from body.
-            document.body.removeChild(video);
+      // Remove the video element from body.
+      document.body.removeChild(video);
 
-            // Reset the video src attribute to re-trigger resource selection for tracks.
-            video.src = getVideoURI('/media/test');
+      // Reset the video src attribute to re-trigger resource selection for tracks.
+      video.src = getVideoURI('/media/test');
 
-            // Append the video element back to the body.
-            document.body.appendChild(video);
+      // Append the video element back to the body.
+      document.body.appendChild(video);
 
-            assert_equals(track.readyState, HTMLTrackElement.ERROR);
+      assert_equals(track.readyState, HTMLTrackElement.ERROR);
 
-            video.onplaying = t.step_func_done();
-            video.play();
-            // The video should start playing.
-        }
-    });
-    </script>
-</video>
+      video.onplaying = t.step_func_done();
+      video.play();
+      // The video should start playing.
+    }
+  });
+</script>