WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
52028
animationTime isn't working on Windows
https://bugs.webkit.org/show_bug.cgi?id=52028
Summary
animationTime isn't working on Windows
Adam Roben (:aroben)
Reported
2011-01-06 15:18:12 PST
The animationTime property isn't working on Windows. Presumably it has been this way since it was added in
r75169
. This is currently causing animations/animation-time.html and fast/dom/Window/window-property-descriptors.html to fail. I've checked in Windows-specific failure results for those tests.
Attachments
Add attachment
proposed patch, testcase, etc.
James Robinson
Comment 1
2011-01-06 15:26:31 PST
That's very odd. I would not expect window.webkitAnimationTime to return undefined, the logic to return a number for that is all in DOMWindow.idl/cpp.
Adam Roben (:aroben)
Comment 2
2011-01-06 15:30:56 PST
<
rdar://problem/8831001
>
James Robinson
Comment 3
2011-01-06 15:34:59 PST
http://build.webkit.org/builders/Windows%20Debug%20%28Build%29/builds/29208
indicates that the Windows Debug builder didn't actually upload a binary after
r75169
, so my guess is tester was using a pre-'169 binary to run the tests added '169. Seems like the tester configuration is broken if it's using old binar
Adam Roben (:aroben)
Comment 4
2011-01-06 15:36:35 PST
(In reply to
comment #3
)
>
http://build.webkit.org/builders/Windows%20Debug%20%28Build%29/builds/29208
indicates that the Windows Debug builder didn't actually upload a binary after
r75169
, so my guess is tester was using a pre-'169 binary to run the tests added '169. Seems like the tester configuration is broken if it's using old binar
Ah! That's due to
bug 51996
.
Adam Roben (:aroben)
Comment 5
2011-01-06 15:45:24 PST
Let's keep this bug open to track removing the Windows-specific results I added once
bug 51996
is fixed.
Adam Roben (:aroben)
Comment 6
2011-01-07 12:45:35 PST
This got fixed in
r75205
.
Adam Roben (:aroben)
Comment 7
2011-01-07 13:36:47 PST
(In reply to
comment #6
)
> This got fixed in
r75205
.
I meant
r75258
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug