Bug 46017 - REGRESSION: alternating animation direction doesn't work on Windows
Summary: REGRESSION: alternating animation direction doesn't work on Windows
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows 7
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords: InRadar, PlatformOnly, Regression
: 45512 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-09-17 17:25 PDT by Mihai Parparita
Modified: 2010-09-20 13:19 PDT (History)
4 users (show)

See Also:


Attachments
Patch (5.72 KB, patch)
2010-09-20 12:28 PDT, Simon Fraser (smfr)
aroben: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mihai Parparita 2010-09-17 17:25:38 PDT
Test case: http://persistent.info/webkit/test-cases/animation-alternate.html

Expected result: gray box moves to the right and then back to the left.

Actual result: gray box moves to the right, jumps back to the starting point, and then moves again to the right.

This happens with both WebKit Window nightly builds and the Chromium canary channel. I haven't had a chance to narrow down the regression range, but it works as expected in the Safari 5.0.2 release but is broken as of r67637.  Chromium noticed this breakage with r67098 (see bug 45512), though I don't see how that would affect Windows only.
Comment 1 Simon Fraser (smfr) 2010-09-17 17:48:45 PDT
This works in my TOT mac build.
Comment 2 Adam Roben (:aroben) 2010-09-20 11:19:08 PDT
Added related test failures to the Skipped file in r67866. http://trac.webkit.org/changeset/67866
Comment 3 Adam Roben (:aroben) 2010-09-20 11:19:43 PDT
<rdar://problem/8453176>
Comment 4 Simon Fraser (smfr) 2010-09-20 12:28:21 PDT
Created attachment 68123 [details]
Patch
Comment 5 Simon Fraser (smfr) 2010-09-20 12:29:38 PDT
*** Bug 45512 has been marked as a duplicate of this bug. ***
Comment 6 Simon Fraser (smfr) 2010-09-20 13:19:40 PDT
http://trac.webkit.org/changeset/67877