testcase for an animated viewBox affecting clip-path.

RESOLVED FIXED in Firefox 52

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: u459114, Assigned: u459114)

Tracking

unspecified
mozilla52
Points:
---

Firefox Tracking Flags

(firefox52 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
According to bug 1289011 comment 26, heycam ask for a test for an animated viewBox="" affecting clip-path.
Comment hidden (mozreview-request)
(Assignee)

Updated

2 years ago
Attachment #8796416 - Flags: review?(cam)

Comment 2

2 years ago
mozreview-review
Comment on attachment 8796416 [details]
Bug 1299760 - a testcase for animated-viewBox affecting clip-path.

https://reviewboard.mozilla.org/r/82288/#review80908

r=me with this change.

::: layout/reftests/svg/clip-path-viewBox-animation.svg:7
(Diff revision 1)
> +  <![CDATA[
> +    window.addEventListener("load", function () {
> +      setTimeout(function () {
> +        document.documentElement.removeAttribute("class");
> +      }, 2000);
> +    }, false);
> +  ]]>

Instead of waiting 2s, please use the same technique that tests in layout/reftests/svg/smil/ do, i.e. using setTimeAndSnapshot from smil-util.js.
Attachment #8796416 - Flags: review?(cam) → review+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 5

2 years ago
Pushed by cku@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7f9e6e421e24
a testcase for animated-viewBox affecting clip-path. r=heycam
(Assignee)

Comment 8

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/adf1eb119bd2998a9b293f962405b8a2d5d29618
Bug 1299760 - a testcase for animated-viewBox affecting clip-path. r=heycam

Comment 9

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/adf1eb119bd2
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.