LayoutTests/http/tests/media/gc-while-network-loading.html - Issue 552303006: Prevent more script-observable cases of HTMLMediaElement GC

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: LayoutTests/http/tests/media/gc-while-network-loading.html

Issue 552303006: Prevent more script-observable cases of HTMLMediaElement GC (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: cleanup Created 6 years, 3 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:

OLD	NEW
(Empty)
	1 <!DOCTYPE html>

	2 <title>GC while networkState is NETWORK_LOADING</title>

	3 <script src="../resources/testharness.js"></script>

	4 <script src="../resources/testharnessreport.js"></script>

	5 <script src="../../media-resources/media-file.js"></script>

	6 <script>

	7 async_test(function(t)

	8 {

	9 var v = document.createElement("video");

	10 v.foo = "bar";

	11 var mediaFile = findMediaFile("video", "../../../media/content/test");

	12 var type = mimeTypeForExtension(mediaFile.split(".").pop());

	13 v.src = "http://127.0.0.1:8000/media/video-throttled-load.cgi?name=" + media File + "&throttle=50&type=" + type;

	14 v.onloadeddata = t.step_func(function()

	15 {

	16 assert_equals(v.networkState, v.NETWORK_LOADING);

	17 // The delaying-the-load-event flag is now false.

	18 // GC async since the event target is the media element.

	19 setTimeout(t.step_func(gcAndAwaitProgress), 0);

	20 });

	21 function gcAndAwaitProgress()

	22 {

	23 v.onprogress = t.step_func(function(e)

	24 {

	25 assert_equals(e.target.foo, "bar");

	26 t.done();

	27 });

	28 v = null;

	29 gc();

	30 }

	31 });

	32 </script>

OLD	NEW