[wpt-sync] Sync PR 18846 - Restore SVGSMILElement::Elapsed()
Categories
(Core :: SVG, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 18846 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/18846
Details from upstream follow.
Fredrik Söderquist <fs@opera.com> wrote:
Restore SVGSMILElement::Elapsed()
When sampling/adding instance times from within the animation engine we
want to use SMILTimeContainer::CurrentDocumentTime() rather than
SMILTimeContainer::Elapsed(). However the latter should still be used
for all event bases and similar. Since most users of
SVGSMILElement::Elapsed() still fall into the "event base" category,
restore it to its original guise and open-code usage of
CurrentDocumentTime() where it's needed.Bug: 1000290
Change-Id: Ie91ae8d5c9c63d03476ba11921777d064b264066Reviewed-on: https://chromium-review.googlesource.com/1783153
WPT-Export-Revision: b35c82e189506b444982a94bcaa4698da25fa9d4
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=62b9afb45de1a8738f1cb1f1838880bbfd7a1b97
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=241b4fb49ea87daf7768fa22a867e3ac21c4de62
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=dc097adcbe885a87e1d4592ea4ad9b201c1a7708
Assignee | ||
Comment 4•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=fb59a14f15444362d1f764826751b05852a4e27a
Assignee | ||
Comment 5•5 years ago
|
||
Ran 1 tests and 1 subtests OK : 1 PASS : 1
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/c2d510d95035 [wpt PR 18846] - Restore SVGSMILElement::Elapsed(), a=testonly
Comment 7•5 years ago
|
||
bugherder |
Description
•