New issue
Advanced search Search tips

Issue 772060 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 772407



Sign in to add a comment

Web Animations: currentTime calculation incorrect in some WPT tests

Project Member Reported by smcgruer@chromium.org, Oct 5 2017

Issue description

This is a bit of a catch-all bug because I haven't been able to root-cause the reason yet. We currently fail some WPT tests that test the currentTime, mostly at boundary conditions.

external/wpt/web-animations/timing-model/animations/updating-the-finished-state.html
  - Updating the finished state when playing before end <-- the currentTime is meant to have moved on when duration changes, but doesn't
  - Updating the finished state when seeking before end <-- the currentTime is meant to move on after a seek but doesn't
  - Updating the finished state when seeking a reversed animation before end <-- the current time is meant to move on after a seek (with playbackRate < 0) but doesn't
 
external/wpt/web-animations/timing-model/animations/set-the-animation-start-time.html
  - Setting the start time updates the finished state <-- setting the start time fails to move the current time past the effect end
 

Comment 1 by shend@chromium.org, Oct 5 2017

Labels: Hotlist-Interop Update-Quarterly
Blocking: 772407
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c48db31689105ce81dcbe640416e45860299c248

commit c48db31689105ce81dcbe640416e45860299c248
Author: Stephen McGruer <smcgruer@chromium.org>
Date: Thu Oct 12 20:34:35 2017

Update crbug links for some failing WPT web-animations tests

Bug:  600248 , 771722, 771751, 771977,  771985 ,  772014 , 772048, 772060, 772076 
Change-Id: Ie08474f751fd45484627c8a52d84db13ca6b39ac
Reviewed-on: https://chromium-review.googlesource.com/702536
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#508447}
[modify] https://crrev.com/c48db31689105ce81dcbe640416e45860299c248/third_party/WebKit/LayoutTests/TestExpectations

It's quite possible these tests are wrong. They pass in Firefox but Firefox's microtask handling is incorrect (https://bugzilla.mozilla.org/show_bug.cgi?id=1193394 -- we're currently in the process of fixing this).

Also, there's a fair bit of unspecified behavior in this area but hopefully I can address that in https://github.com/whatwg/html/issues/707 soon.
(Which is to say that we'll likely fix any broken tests in Firefox in the next week or two and then they should be upstreamed to web-platform-tests.)
Cc: smcgruer@chromium.org
Owner: ----
Status: Available (was: Assigned)
Not planning to work on this in the near future, dropping to Available.

Sign in to add a comment