<?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>33370</bug_id>
          
          <creation_ts>2010-01-08 02:51:40 -0800</creation_ts>
          <short_desc>animations/change-keyframes.html failed on Tiger Bot</short_desc>
          <delta_ts>2010-01-14 17:52:51 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>33289</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>cmarrin</cc>
    
    <cc>ojan</cc>
    
    <cc>zimmermann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>178586</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-01-08 02:51:40 -0800</bug_when>
    <thetext>animations/change-keyframes.html failed on Tiger Bot

This is the first time I&apos;ve noticed this failure:
http://build.webkit.org/results/Tiger%20Intel%20Release/r52980%20(7550)/results.html

--- layout-test-results/animations/change-keyframes-expected.txt	2010-01-08 02:35:58.000000000 -0800
+++ layout-test-results/animations/change-keyframes-actual.txt	2010-01-08 02:35:58.000000000 -0800
@@ -1,4 +1,4 @@
 This test performs an animation of the left property and makes sure it is animating. Then it stops the animation, changes the keyframes to an animation of the top property, restarts the animation and makes sure top is animating.
-PASS - &quot;left&quot; property for &quot;box&quot; element at 0.5s saw something close to: 200
-PASS - &quot;top&quot; property for &quot;box&quot; element at 1s saw something close to: 100
+FAIL - &quot;left&quot; property for &quot;box&quot; element at 0.5s expected: 200 but saw: 0
+FAIL - &quot;top&quot; property for &quot;box&quot; element at 1s expected: 100 but saw: 0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>179198</commentid>
    <comment_count>1</comment_count>
    <who name="Nikolas Zimmermann">zimmermann</who>
    <bug_when>2010-01-10 16:59:21 -0800</bug_when>
    <thetext>Fails differently now:

http://build.webkit.org/results/Tiger%20Intel%20Release/r53055%20(7612)/animations/change-keyframes-pretty-diff.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>179214</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-01-10 19:06:38 -0800</bug_when>
    <thetext>Our buildbot instance is set up to purge results after a while, so we need to copy failure diffs into bugs:
--- layout-test-results/animations/change-keyframes-expected.txt	2010-01-10 16:41:40.000000000 -0800
+++ layout-test-results/animations/change-keyframes-actual.txt	2010-01-10 16:41:40.000000000 -0800
@@ -1,4 +1,4 @@
 This test performs an animation of the left property and makes sure it is animating. Then it stops the animation, changes the keyframes to an animation of the top property, restarts the animation and makes sure top is animating.
-PASS - &quot;left&quot; property for &quot;box&quot; element at 0.5s saw something close to: 200
-PASS - &quot;top&quot; property for &quot;box&quot; element at 1s saw something close to: 100
+FAIL - &quot;left&quot; property for &quot;box&quot; element at 0.5s expected: 200 but saw: 259
+FAIL - &quot;top&quot; property for &quot;box&quot; element at 1s expected: 100 but saw: 70</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180997</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-01-14 16:29:14 -0800</bug_when>
    <thetext>Another failure, different diff:
http://build.webkit.org/results/Tiger%20Intel%20Release/r53296%20(7779)/animations/change-keyframes-diffs.txt
--- layout-test-results/animations/change-keyframes-expected.txt	2010-01-14 16:15:08.000000000 -0800
+++ layout-test-results/animations/change-keyframes-actual.txt	2010-01-14 16:15:08.000000000 -0800
@@ -1,4 +1,4 @@
 This test performs an animation of the left property and makes sure it is animating. Then it stops the animation, changes the keyframes to an animation of the top property, restarts the animation and makes sure top is animating.
-PASS - &quot;left&quot; property for &quot;box&quot; element at 0.5s saw something close to: 200
+FAIL - &quot;left&quot; property for &quot;box&quot; element at 0.5s expected: 200 but saw: 252
 PASS - &quot;top&quot; property for &quot;box&quot; element at 1s saw something close to: 100</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>181043</commentid>
    <comment_count>4</comment_count>
      <attachid>46625</attachid>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2010-01-14 17:51:01 -0800</bug_when>
    <thetext>Created attachment 46625
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>181045</commentid>
    <comment_count>5</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2010-01-14 17:52:51 -0800</bug_when>
    <thetext>Committed r53310: &lt;http://trac.webkit.org/changeset/53310&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>46625</attachid>
            <date>2010-01-14 17:51:01 -0800</date>
            <delta_ts>2010-01-14 17:51:47 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-33370-20100114175100.patch</filename>
            <type>text/plain</type>
            <size>2732</size>
            <attacher name="Ojan Vafai">ojan</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCBkZDg2NjYyLi5mZTQwMzhlIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTAtMDEt
MTQgIE9qYW4gVmFmYWkgIDxvamFuQGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBhbmltYXRpb25zL2NoYW5nZS1rZXlmcmFtZXMu
aHRtbCBmYWlsZWQgb24gVGlnZXIgQm90CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0zMzM3MAorCisgICAgICAgIEluY3JlYXNlIHRvbGVyYW5jZXMuIFRo
ZXkncmUgc3RpbGwgdGVzdGluZyB0aGUgc3RhdGVkIGdvYWwgb2YgdGhlIHRlc3QuCisKKyAgICAg
ICAgKiBhbmltYXRpb25zL2NoYW5nZS1rZXlmcmFtZXMuaHRtbDoKKyAgICAgICAgKiB0cmFuc2l0
aW9ucy9jYW5jZWwtdHJhbnNpdGlvbi5odG1sOgorCiAyMDEwLTAxLTE0ICBHdXN0YXZvIE5vcm9u
aGEgU2lsdmEgIDxnbnNAZ25vbWUub3JnPgogCiAgICAgICAgIFNraXAgbmV3IHRlc3QgdGhhdCBy
ZXF1aXJlcyBuZXcgZnVuY3Rpb25hbGl0eSBpbgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvYW5p
bWF0aW9ucy9jaGFuZ2Uta2V5ZnJhbWVzLmh0bWwgYi9MYXlvdXRUZXN0cy9hbmltYXRpb25zL2No
YW5nZS1rZXlmcmFtZXMuaHRtbAppbmRleCBiNzAxNmQyLi5jYTY2NDg4IDEwMDY0NAotLS0gYS9M
YXlvdXRUZXN0cy9hbmltYXRpb25zL2NoYW5nZS1rZXlmcmFtZXMuaHRtbAorKysgYi9MYXlvdXRU
ZXN0cy9hbmltYXRpb25zL2NoYW5nZS1rZXlmcmFtZXMuaHRtbApAQCAtMTksOCArMTksOCBAQAog
ICAgIH0KICAgICBALXdlYmtpdC1rZXlmcmFtZXMgImFuaW0iIHsKICAgICAgICAgZnJvbSB7IGxl
ZnQ6IDEwMHB4OyB9Ci0gICAgICAgIDQwJSAgeyBsZWZ0OiAyMDBweDsgfQotICAgICAgICA2MCUg
IHsgbGVmdDogMjAwcHg7IH0KKyAgICAgICAgMTAlICB7IGxlZnQ6IDIwMHB4OyB9CisgICAgICAg
IDkwJSAgeyBsZWZ0OiAyMDBweDsgfQogICAgICAgICB0byAgIHsgbGVmdDogMzAwcHg7IH0KICAg
ICB9CiAgICAgPC9zdHlsZT4KQEAgLTU2LDcgKzU2LDcgQEAKICAgICAgICAga2V5ZnJhbWVzLmRl
bGV0ZVJ1bGUoIjEwMCUiKTsKICAgICAgICAga2V5ZnJhbWVzLmluc2VydFJ1bGUoIjAlIHsgdG9w
OiA1MHB4OyB9Iik7CiAgICAgICAgIGtleWZyYW1lcy5pbnNlcnRSdWxlKCIxMCUgeyB0b3A6IDEw
MHB4OyB9Iik7Ci0gICAgICAgIGtleWZyYW1lcy5pbnNlcnRSdWxlKCI1MCUgeyB0b3A6IDEwMHB4
OyB9Iik7CisgICAgICAgIGtleWZyYW1lcy5pbnNlcnRSdWxlKCI5MCUgeyB0b3A6IDEwMHB4OyB9
Iik7CiAgICAgICAgIGtleWZyYW1lcy5pbnNlcnRSdWxlKCIxMDAlIHsgdG9wOiAxNTBweDsgfSIp
OwogICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnYm94Jykuc3R5bGUud2Via2l0QW5p
bWF0aW9uTmFtZSA9ICJhbmltIjsKICAgICB9CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy90cmFu
c2l0aW9ucy9jYW5jZWwtdHJhbnNpdGlvbi5odG1sIGIvTGF5b3V0VGVzdHMvdHJhbnNpdGlvbnMv
Y2FuY2VsLXRyYW5zaXRpb24uaHRtbAppbmRleCA4MzM3NWVjLi45M2RlNTdmIDEwMDY0NAotLS0g
YS9MYXlvdXRUZXN0cy90cmFuc2l0aW9ucy9jYW5jZWwtdHJhbnNpdGlvbi5odG1sCisrKyBiL0xh
eW91dFRlc3RzL3RyYW5zaXRpb25zL2NhbmNlbC10cmFuc2l0aW9uLmh0bWwKQEAgLTY4LDcgKzY4
LDcgQEAKICAgICAgICAgewogICAgICAgICAgICAgdmFyIGxlZnQgPSBwYXJzZUZsb2F0KHdpbmRv
dy5nZXRDb21wdXRlZFN0eWxlKGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdhJykpLmxlZnQpOwog
ICAgICAgICAgICAgcmVzdWx0ID0gImxlZnQ6IjsKLSAgICAgICAgICAgIGlmICghaXNFcXVhbChs
ZWZ0LCAyNTAsIDMwKSkKKyAgICAgICAgICAgIGlmICghaXNFcXVhbChsZWZ0LCAyNTAsIDUwKSkK
ICAgICAgICAgICAgICAgICByZXN1bHQgKz0gIjxzcGFuIHN0eWxlPSdjb2xvcjpyZWQnPkZBSUwo
d2FzOiIgKyBsZWZ0ICsgIiwgcy9iOjE1MCk8L3NwYW4+IjsKICAgICAgICAgICAgIGVsc2UKICAg
ICAgICAgICAgICAgICByZXN1bHQgKz0gIjxzcGFuIHN0eWxlPSdjb2xvcjpncmVlbic+UEFTUzwv
c3Bhbj4iOwpAQCAtNzgsNyArNzgsNyBAQAogICAgICAgICAgICAgdmFyIHRyYW5zZm9ybSA9IHdp
bmRvdy5nZXRDb21wdXRlZFN0eWxlKGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdiJykpLndlYmtp
dFRyYW5zZm9ybTsKICAgICAgICAgICAgIHRyYW5zZm9ybSA9IHRyYW5zZm9ybS5zcGxpdCgiKCIp
OwogICAgICAgICAgICAgdHJhbnNmb3JtID0gdHJhbnNmb3JtWzFdLnNwbGl0KCIsIik7Ci0gICAg
ICAgICAgICBpZiAoIWlzRXF1YWwodHJhbnNmb3JtWzRdLCAyMDAsIDMwKSkKKyAgICAgICAgICAg
IGlmICghaXNFcXVhbCh0cmFuc2Zvcm1bNF0sIDIwMCwgNTApKQogICAgICAgICAgICAgICAgIHJl
c3VsdCArPSAiPHNwYW4gc3R5bGU9J2NvbG9yOnJlZCc+RkFJTCh3YXM6IiArIHRyYW5zZm9ybVs0
XSArICIsIHMvYjo1MCk8L3NwYW4+IjsKICAgICAgICAgICAgIGVsc2UKICAgICAgICAgICAgICAg
ICByZXN1bHQgKz0gIjxzcGFuIHN0eWxlPSdjb2xvcjpncmVlbic+UEFTUzwvc3Bhbj4iOwo=
</data>
<flag name="review"
          id="28927"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>