LayoutTests/svg/as-background-image/animated-svg-animation-control.html - Issue 206033005: Rewind the timeline for SVG-in-<img> when resetAnimation() is called

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/as-background-image/animated-svg-animation-control.html

Issue 206033005: Rewind the timeline for SVG-in-<img> when resetAnimation() is called (Closed) Base URL: svn://svn.chromium.org/blink/trunk

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 unified diff | Download patch | Annotate | Revision Log

« no previous file with comments | « no previous file | LayoutTests/svg/as-background-image/animated-svg-animation-control-expected.html » ('j') | LayoutTests/svg/as-background-image/animated-svg-animation-control-expected.html » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <!DOCTYPE html>

	2 <title>SVG-in-<img>: Animation resets when all references are removed.</ti tle>

	3 <div style="width: 100px; height: 100px;"></div>

	4 <script>

	5 if (window.testRunner)

	6 testRunner.waitUntilDone();

	7 var div = document.querySelector('div');

	8 var imgurl = "url(\"data:image/svg+xml," +

	9 "<svg xmlns='http://www.w3.org/2000/svg' width='100' height='100'>" +

	10 "<rect width='100' height='100' fill='blue'>" +

	11 "<set attributeName='fill' to='green' dur='0.1s'/>" +

	12 "</rect>" +

	13 "</svg>\")";
	kouhei (in TOK) 2014/03/20 15:27:29 Maybe we can have this as a svg file in LayoutTest Maybe we can have this as a svg file in LayoutTests/svg/as-image/resources/ dir. fs 2014/03/20 17:02:17 Done. (But to as-background-image/resources/ - I g Show quoted text On 2014/03/20 15:27:29, kouhei wrote: > Maybe we can have this as a svg file in LayoutTests/svg/as-image/resources/ dir. Done. (But to as-background-image/resources/ - I generally prefer to keep tests as self-contained as possible - and runnable in stock-browsers if possible.)
	14 div.style.background = imgurl;
	kouhei (in TOK) 2014/03/20 15:27:29 There are LayoutTests/svg/as-image/ testing svg in There are LayoutTests/svg/as-image/ testing svg inside <img>. Maybe migrate this there as <img src='foobar.svg'> test? fs 2014/03/20 17:02:17 The bug was reported against 'background-image' so Show quoted text On 2014/03/20 15:27:29, kouhei wrote: > There are LayoutTests/svg/as-image/ testing svg inside <img>. Maybe migrate this > there as <img src='foobar.svg'> test? The bug was reported against 'background-image' so that's why the test was put there. It ought to be possible to have a <img>-based test too (but the difference in code-paths shouldn't matter.) I heard that TCs don't mind quantity, so I'll add one there as well.
	15 setTimeout(function() {

	16 div.style.background = "";

	17 setTimeout(function() {

	18 div.style.background = imgurl;

	19 if (window.testRunner)

	20 testRunner.notifyDone();

	21 }, 10);

	22 }, 150);

	23 </script>

	24

OLD	NEW