web platform tests for composite: 'accumulate'

RESOLVED FIXED in Firefox 55

Status

()

Core
DOM: Animation
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: hiro, Assigned: hiro)

Tracking

Trunk
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

()

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

Attachments

(4 attachments)

(Assignee)

Description

5 months ago
It would be nice to have test cases before we implement bug 1353202.
(Assignee)

Comment 1

5 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a6e172ca19fd82c53539cc73ee426002b79e3b16
Duplicate of this bug: 1364797
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 7

5 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a9d88d6eb1e495ed6ac3b870b196304592cabde0
(Assignee)

Comment 8

5 months ago
mozreview-review
Comment on attachment 8867598 [details]
Bug 1364795 - Update web-platform tests meta data.

https://reviewboard.mozilla.org/r/139154/#review142382
Attachment #8867598 - Flags: review?(hikezoe) → review+
(Assignee)

Comment 9

5 months ago
mozreview-review
Comment on attachment 8867599 [details]
Bug 1364795 - Untabify in testing/web-platform/tests/web-animations/animation-model/.

https://reviewboard.mozilla.org/r/139156/#review142384

Hmm, MozReview does not show up whitespace changes in property-list.js. Why?
Comment on attachment 8867600 [details]
Bug 1364795 Fix CompositeOperation property name.

https://reviewboard.mozilla.org/r/139158/#review142388
Attachment #8867600 - Flags: review?(jeremychen) → review+

Comment 11

5 months ago
mozreview-review
Comment on attachment 8867599 [details]
Bug 1364795 - Untabify in testing/web-platform/tests/web-animations/animation-model/.

https://reviewboard.mozilla.org/r/139156/#review142386

Thanks.
Attachment #8867599 - Flags: review?(mantaroh) → review+
Comment hidden (mozreview-request)

Comment 13

5 months ago
mozreview-review
Comment on attachment 8867601 [details]
Bug 1364795 - Add tests for accumulation per property.

https://reviewboard.mozilla.org/r/139160/#review142404

LGTM
Attachment #8867601 - Flags: review?(boris.chiou) → review+
(Assignee)

Comment 14

5 months ago
Thank you all for the quick review!

Comment 15

5 months ago
Pushed by hikezoe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4f155fb00e0c
Update web-platform tests meta data. r=hiro
https://hg.mozilla.org/integration/autoland/rev/3f1a530b2469
Untabify in testing/web-platform/tests/web-animations/animation-model/. r=mantaroh
https://hg.mozilla.org/integration/autoland/rev/173009f8a24f
Fix CompositeOperation property name. r=jeremychen
https://hg.mozilla.org/integration/autoland/rev/72dbfa516e3e
Add tests for accumulation per property. r=boris
Blocks: 1353202

Comment 16

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/4f155fb00e0c
https://hg.mozilla.org/mozilla-central/rev/3f1a530b2469
https://hg.mozilla.org/mozilla-central/rev/173009f8a24f
https://hg.mozilla.org/mozilla-central/rev/72dbfa516e3e
Status: ASSIGNED → RESOLVED
Last Resolved: 5 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.