LayoutTests/svg/dom/SVGScriptElement/script-external-no-multiple-load.html - Issue 1141683003: Don't dispatch two 'load' events for external SVGScriptElement loads

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/svg/dom/SVGScriptElement/script-external-no-multiple-load.html

Issue 1141683003: Don't dispatch two 'load' events for external SVGScriptElement loads (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Created 5 years, 7 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 <script src=../../../resources/js-test.js></script>

	3 <script>

	4 window.jsTestIsAsync = true;

	5

	6 var loadEventsFired = 0;

	7

	8 onload = function() {

	9 var newExternalScript = document.createElementNS("http://www.w3.org/2000/svg ", "script");

	10 newExternalScript.onload = function() {

	11 loadEventsFired++;

	12 setTimeout(function() {
	fs 2015/05/19 07:25:40 Not very fond of this, but did manage to come up w Not very fond of this, but did manage to come up with anything better to latch on to... Suggestions welcome. pdr. 2015/05/19 18:18:26 I think this is fine. setTimeout is our hook for w Show quoted text On 2015/05/19 at 07:25:40, fs wrote: > Not very fond of this, but did manage to come up with anything better to latch on to... Suggestions welcome. I think this is fine. setTimeout is our hook for waiting for pending tasks.
	13 shouldBe("loadEventsFired", "1");

	14 finishJSTest();

	15 }, 0);

	16 };

	17 newExternalScript.setAttributeNS("http://www.w3.org/1999/xlink", "href", "re sources/script-load.js");

	18 document.querySelector("svg").appendChild(newExternalScript);

	19 }

	20 </script>

	21 <svg>

	22 </svg>

OLD	NEW

« no previous file with comments | « no previous file | LayoutTests/svg/dom/SVGScriptElement/script-external-no-multiple-load-expected.txt » ('j') | no next file with comments »