Bug 52028 - animationTime isn't working on Windows
Summary: animationTime isn't working on Windows
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Nobody
URL: http://build.webkit.org/results/Windo...
Keywords: InRadar, LayoutTestFailure, PlatformOnly
Depends on: 51996
Blocks:
  Show dependency tree / graph
 
Reported: 2011-01-06 15:18 PST by Adam Roben (:aroben)
Modified: 2011-01-07 13:36 PST (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 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.
Comment 1 James Robinson 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.
Comment 2 Adam Roben (:aroben) 2011-01-06 15:30:56 PST 
<rdar://problem/8831001>
Comment 3 James Robinson 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
Comment 4 Adam Roben (:aroben) 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.
Comment 5 Adam Roben (:aroben) 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.
Comment 6 Adam Roben (:aroben) 2011-01-07 12:45:35 PST 
This got fixed in r75205.
Comment 7 Adam Roben (:aroben) 2011-01-07 13:36:47 PST 
(In reply to comment #6)
> This got fixed in r75205.

I meant r75258.