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: | |||
Adam Roben (:aroben)
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
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)
<rdar://problem/8831001>
James Robinson
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)
(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)
Let's keep this bug open to track removing the Windows-specific results I added once bug 51996 is fixed.
Adam Roben (:aroben)
This got fixed in r75205.
Adam Roben (:aroben)
(In reply to comment #6)
> This got fixed in r75205.
I meant r75258.