1343415 - Add SMIL test xlink:href for outer HTML Element.

Status: RESOLVED FIXED

(Core :: SVG, enhancement)

Description

[Mantaroh Yoshinaga[:mantaroh]]

Attachment: WIP: add smil test with xlink:href attribute.

We can specify the SMIL's animate tag with xlink:href attribute, and then we can specify outer html element like following:

<div id="target"></div>
<svg xmlns:xlink="http://www.w3.org/1999/xlink">
 <animate xlink:href="#target" attributeName="width" from.....>
</svg>

In the gecko, some code support this behavior.

For Example: https://dxr.mozilla.org/mozilla-central/rev/e1135c6fdc9bcd80d38f7285b269e030716dcb72/dom/smil/nsSMILAnimationController.cpp#703

But, we don't have this test.
I will need to this test since we will change around this code.(see bug 1062106)

Comment 1

[Mantaroh Yoshinaga[:mantaroh]]

Attachment: Bug 1343415 - Add test for SMIL with xmil:href attribute which specifying outer element.

Review commit: https://reviewboard.mozilla.org/r/116140/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/116140/

Comment 2

[Mantaroh Yoshinaga[:mantaroh]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f98bce2adacd387f05027d98d3145b82431412e1

Comment 3

[Brian Birtles (:birtles)]

Comment on attachment 8842263 [details]
Bug 1343415 - Add test for SMIL with xmil:href attribute which specifying outer element.

https://reviewboard.mozilla.org/r/116140/#review117724

::: dom/smil/test/test_smilWithXlink.xhtml:34
(Diff revision 1)
> +
> +  var cs = getComputedStyle(target);
> +  is(cs.width, "0px",
> +     "SMIL should affect outer element when animation is started.");
> +
> +  svg.setCurrentTime(10);
> +
> +  cs = getComputedStyle(target);
> +  is(cs.width, "200px",
> +     "SMIL should affect outer element when animation is ended");

Nit: I think checking one time value would be enough?

e.g. we could just do:

svg.pauseAnimations();
svg.setCurrentTime(5);

var cs = getComputedStyle(target);
is(cs.width, "50px", ...);

And that would be enough?

Comment 4

[Mantaroh Yoshinaga[:mantaroh]]

Comment on attachment 8842263 [details]
Bug 1343415 - Add test for SMIL with xmil:href attribute which specifying outer element.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/116140/diff/1-2/

Comment 5

[Mantaroh Yoshinaga[:mantaroh]]

Comment on attachment 8842263 [details]
Bug 1343415 - Add test for SMIL with xmil:href attribute which specifying outer element.

https://reviewboard.mozilla.org/r/116140/#review117724

> Nit: I think checking one time value would be enough?
> 
> e.g. we could just do:
> 
> svg.pauseAnimations();
> svg.setCurrentTime(5);
> 
> var cs = getComputedStyle(target);
> is(cs.width, "50px", ...);
> 
> And that would be enough?

Thanks Brian,

I removed extra check.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/98785831bbb8
Add test for SMIL with xmil:href attribute which specifying outer element. r=birtles

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/98785831bbb8