Bug 52028

Summary:	animationTime isn't working on Windows
Product:	WebKit	Reporter:	Adam Roben (:aroben) <aroben>
Component:	DOM	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	jamesr, simon.fraser
Priority:	P2	Keywords:	InRadar, LayoutTestFailure, PlatformOnly
Version:	528+ (Nightly build)
Hardware:	PC
OS:	Windows XP
URL:	http://build.webkit.org/results/Windows%20XP%20Debug%20(Tests)/r75196%20(23989)/animations/animation-time-pretty-diff.html
Bug Depends on:	51996
Bug Blocks:

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.