<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>52028</bug_id>
          
          <creation_ts>2011-01-06 15:18:12 -0800</creation_ts>
          <short_desc>animationTime isn&apos;t working on Windows</short_desc>
          <delta_ts>2011-01-07 13:36:47 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>DOM</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://build.webkit.org/results/Windows%20XP%20Debug%20(Tests)/r75196%20(23989)/animations/animation-time-pretty-diff.html</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, LayoutTestFailure, PlatformOnly</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>51996</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Roben (:aroben)">aroben</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>jamesr</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>330475</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-01-06 15:18:12 -0800</bug_when>
    <thetext>The animationTime property isn&apos;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&apos;ve checked in Windows-specific failure results for those tests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>330486</commentid>
    <comment_count>1</comment_count>
    <who name="James Robinson">jamesr</who>
    <bug_when>2011-01-06 15:26:31 -0800</bug_when>
    <thetext>That&apos;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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>330496</commentid>
    <comment_count>2</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-01-06 15:30:56 -0800</bug_when>
    <thetext>&lt;rdar://problem/8831001&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>330501</commentid>
    <comment_count>3</comment_count>
    <who name="James Robinson">jamesr</who>
    <bug_when>2011-01-06 15:34:59 -0800</bug_when>
    <thetext>http://build.webkit.org/builders/Windows%20Debug%20%28Build%29/builds/29208 indicates that the Windows Debug builder didn&apos;t actually upload a binary after r75169, so my guess is tester was using a pre-&apos;169 binary to run the tests added &apos;169.  Seems like the tester configuration is broken if it&apos;s using old binar</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>330503</commentid>
    <comment_count>4</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-01-06 15:36:35 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; http://build.webkit.org/builders/Windows%20Debug%20%28Build%29/builds/29208 indicates that the Windows Debug builder didn&apos;t actually upload a binary after r75169, so my guess is tester was using a pre-&apos;169 binary to run the tests added &apos;169.  Seems like the tester configuration is broken if it&apos;s using old binar

Ah! That&apos;s due to bug 51996.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>330510</commentid>
    <comment_count>5</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-01-06 15:45:24 -0800</bug_when>
    <thetext>Let&apos;s keep this bug open to track removing the Windows-specific results I added once bug 51996 is fixed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>330934</commentid>
    <comment_count>6</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-01-07 12:45:35 -0800</bug_when>
    <thetext>This got fixed in r75205.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>330975</commentid>
    <comment_count>7</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-01-07 13:36:47 -0800</bug_when>
    <thetext>(In reply to comment #6)
&gt; This got fixed in r75205.

I meant r75258.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>