33461 2010-01-10 21:03:00 -0800 animations/combo-transform-translate+scale.html failed on Snow Leopard Debug Bot 2010-01-15 09:46:15 -0800 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) PC OS X 10.5 RESOLVED FIXED P2 Normal --- 33292 1 eric webkit-unassigned cmarrin ojan oldest_to_newest 179258 0 eric 2010-01-10 21:03:00 -0800 animations/combo-transform-translate+scale.html failed on Snow Leopard Debug Bot I suspect this is just a flakey test. http://build.webkit.org/results/SnowLeopard%20Intel%20Leaks/r53058%20(3325)/animations/combo-transform-translate+scale-diffs.txt --- layout-test-results/animations/combo-transform-translate+scale-expected.txt 2010-01-10 19:45:06.000000000 -0800 +++ layout-test-results/animations/combo-transform-translate+scale-actual.txt 2010-01-10 19:45:06.000000000 -0800 @@ -1,2 +1,2 @@ This test performs an animation of the translate() and scale() operators. It animates over 2 seconds. At 0.5 and 1 second it takes a snapshot and expects the result to be within range. -PASS +FAIL('d' was:2.71446, expected:2.5) http://trac.webkit.org/browser/trunk/LayoutTests/animations/combo-transform-translate+scale.html 180651 1 eric 2010-01-14 03:00:00 -0800 Failed again just now: http://build.webkit.org/results/SnowLeopard%20Intel%20Leaks/r53248%20(3417)/animations/combo-transform-translate+scale-diffs.txt different diff than last time: --- layout-test-results/animations/combo-transform-translate+scale-expected.txt 2010-01-14 01:40:27.000000000 -0800 +++ layout-test-results/animations/combo-transform-translate+scale-actual.txt 2010-01-14 01:40:27.000000000 -0800 @@ -1,2 +1,2 @@ This test performs an animation of the translate() and scale() operators. It animates over 2 seconds. At 0.5 and 1 second it takes a snapshot and expects the result to be within range. -PASS +FAIL('f' was:111, expected:100) 181057 2 46628 ojan 2010-01-14 18:07:06 -0800 Created attachment 46628 Patch 181065 3 46628 eric 2010-01-14 18:30:48 -0800 Comment on attachment 46628 Patch Don't you want to start the timeouts from the startedTransform callback? 181069 4 ojan 2010-01-14 18:37:54 -0800 (In reply to comment #3) > (From update of attachment 46628 [details]) > Don't you want to start the timeouts from the startedTransform callback? It already does. The only timeout I'm removing is the one that writes out the already calculated result and calls notifyDone. The timeouts that do measurement are still started in the webkitAnimationStart handler. 181070 5 46628 eric 2010-01-14 18:40:15 -0800 Comment on attachment 46628 Patch OK. 181254 6 ojan 2010-01-15 09:46:15 -0800 Committed r53333: <http://trac.webkit.org/changeset/53333> 46628 2010-01-14 18:07:06 -0800 2010-01-14 18:40:14 -0800 Patch bug-33461-20100114180705.patch text/plain 4452 ojan ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCBkNDU0MjBmLi4yMzdmYmQ1IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTcgQEAKIDIwMTAtMDEt MTQgIE9qYW4gVmFmYWkgIDxvamFuQGNocm9taXVtLm9yZz4KIAorICAgICAgICBSZXZpZXdlZCBi eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBhbmltYXRpb25zL2NvbWJvLXRyYW5zZm9ybS10 cmFuc2xhdGUrc2NhbGUuaHRtbCBmYWlsZWQgb24gU25vdyBMZW9wYXJkIERlYnVnIEJvdAorICAg ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzM0NjEKKworICAg ICAgICBNYWtlIHRlc3QgaGFsZiBhcyBsb25nLiBSZW1vdmUgYSBzZXRUaW1lb3V0IGFuZCB0d2Vh ayB0b2xlcmFuY2VzIHRvCisgICAgICAgIHJlZHVjZSBmbGFraW5lc3MuCisKKyAgICAgICAgKiBh bmltYXRpb25zL2NvbWJvLXRyYW5zZm9ybS10cmFuc2xhdGUrc2NhbGUuaHRtbDoKKworMjAxMC0w MS0xNCAgT2phbiBWYWZhaSAgPG9qYW5AY2hyb21pdW0ub3JnPgorCiAgICAgICAgIFJldmlld2Vk IGJ5IEVyaWMgU2VpZGVsLgogCiAgICAgICAgIGFuaW1hdGlvbnMvY2hhbmdlLWtleWZyYW1lcy5o dG1sIGZhaWxlZCBvbiBUaWdlciBCb3QKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2FuaW1hdGlv bnMvY29tYm8tdHJhbnNmb3JtLXRyYW5zbGF0ZStzY2FsZS1leHBlY3RlZC50eHQgYi9MYXlvdXRU ZXN0cy9hbmltYXRpb25zL2NvbWJvLXRyYW5zZm9ybS10cmFuc2xhdGUrc2NhbGUtZXhwZWN0ZWQu dHh0CmluZGV4IDkxYzlkNGYuLjY1ZWM2ODAgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL2FuaW1h dGlvbnMvY29tYm8tdHJhbnNmb3JtLXRyYW5zbGF0ZStzY2FsZS1leHBlY3RlZC50eHQKKysrIGIv TGF5b3V0VGVzdHMvYW5pbWF0aW9ucy9jb21iby10cmFuc2Zvcm0tdHJhbnNsYXRlK3NjYWxlLWV4 cGVjdGVkLnR4dApAQCAtMSwyICsxLDIgQEAKLVRoaXMgdGVzdCBwZXJmb3JtcyBhbiBhbmltYXRp b24gb2YgdGhlIHRyYW5zbGF0ZSgpIGFuZCBzY2FsZSgpIG9wZXJhdG9ycy4gSXQgYW5pbWF0ZXMg b3ZlciAyIHNlY29uZHMuIEF0IDAuNSBhbmQgMSBzZWNvbmQgaXQgdGFrZXMgYSBzbmFwc2hvdCBh bmQgZXhwZWN0cyB0aGUgcmVzdWx0IHRvIGJlIHdpdGhpbiByYW5nZS4KK1RoaXMgdGVzdCBwZXJm b3JtcyBhbiBhbmltYXRpb24gb2YgdGhlIHRyYW5zbGF0ZSgpIGFuZCBzY2FsZSgpIG9wZXJhdG9y cy4gSXQgYW5pbWF0ZXMgb3ZlciAxIHNlY29uZHMuIEF0IDAuMjUgYW5kIDAuNSBzZWNvbmRzIGl0 IHRha2VzIGEgc25hcHNob3QgYW5kIGV4cGVjdHMgdGhlIHJlc3VsdCB0byBiZSB3aXRoaW4gcmFu Z2UuCiBQQVNTCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9hbmltYXRpb25zL2NvbWJvLXRyYW5z Zm9ybS10cmFuc2xhdGUrc2NhbGUuaHRtbCBiL0xheW91dFRlc3RzL2FuaW1hdGlvbnMvY29tYm8t dHJhbnNmb3JtLXRyYW5zbGF0ZStzY2FsZS5odG1sCmluZGV4IDM2MmI2ZTguLmQzMzBjMTQgMTAw NjQ0Ci0tLSBhL0xheW91dFRlc3RzL2FuaW1hdGlvbnMvY29tYm8tdHJhbnNmb3JtLXRyYW5zbGF0 ZStzY2FsZS5odG1sCisrKyBiL0xheW91dFRlc3RzL2FuaW1hdGlvbnMvY29tYm8tdHJhbnNmb3Jt LXRyYW5zbGF0ZStzY2FsZS5odG1sCkBAIC0xMCw3ICsxMCw3IEBACiAgICAgICBoZWlnaHQ6IDEw MHB4OwogICAgICAgd2lkdGg6IDEwMHB4OwogICAgICAgYmFja2dyb3VuZC1jb2xvcjogYmx1ZTsK LSAgICAgIC13ZWJraXQtYW5pbWF0aW9uLWR1cmF0aW9uOiAyczsKKyAgICAgIC13ZWJraXQtYW5p bWF0aW9uLWR1cmF0aW9uOiAxczsKICAgICAgIC13ZWJraXQtYW5pbWF0aW9uLXRpbWluZy1mdW5j dGlvbjogbGluZWFyOwogICAgICAgLXdlYmtpdC1hbmltYXRpb24tbmFtZTogImFuaW0iOwogICAg IH0KQEAgLTI1LDExICsyNSwxMSBAQAogICAgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIud2FpdFVu dGlsRG9uZSgpOwogICAgIH0KICAgICAKLSAgICByZXN1bHQgPSAiUEFTUyI7CisgICAgcmVzdWx0 ID0gIiI7CiAKICAgICBjb25zdCBleHBlY3RlZCA9ICBbIFsgMS4yNSwwLDAsMS43NSwyNSw1MCBd LAogICAgICAgICAgICAgICAgICAgICAgICAgWyAxLjUsMCwwLDIuNSw1MCwxMDAgXSBdOwotICAg IGNvbnN0IHRvbGVyYW5jZSA9IFsgMC4yLDAuMiwwLjIsMC4yLDUsMTAgXTsKKyAgICBjb25zdCB0 b2xlcmFuY2UgPSBbIDAuMiwwLjIsMC4yLDAuMyw1LDE1IF07CiAgICAgY29uc3QgcHJvcCA9IFsg ImEiLCAiYiIsICJjIiwgImQiLCAiZSIsICJmIiBdOwogICAgIAogICAgIGZ1bmN0aW9uIGlzRXF1 YWwoYWN0dWFsLCBkZXNpcmVkLCB0b2xlcmFuY2UpCkBAIC0zOCwyOSArMzgsMjggQEAKICAgICAg ICAgcmV0dXJuIGRpZmYgPD0gdG9sZXJhbmNlOwogICAgIH0KICAgICAKKyAgICB2YXIgbnVtU25h cHNob3RzID0gMDsKKyAgICAKICAgICBmdW5jdGlvbiBzbmFwc2hvdCh3aGljaCkKICAgICB7Ci0g ICAgICAgIGlmIChyZXN1bHQgIT0gIlBBU1MiKQotICAgICAgICAgICAgcmV0dXJuOwotICAgICAg ICAKICAgICAgICAgdmFyIHQgPSBuZXcgV2ViS2l0Q1NTTWF0cml4KHdpbmRvdy5nZXRDb21wdXRl ZFN0eWxlKGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdib3gnKSkud2Via2l0VHJhbnNmb3JtKTsK LSAgICAgICAgZm9yIChpID0gMDsgaSA8IDY7ICsraSkKLSAgICAgICAgICAgIGlmICghaXNFcXVh bCh0W3Byb3BbaV1dLCBleHBlY3RlZFt3aGljaF1baV0sIHRvbGVyYW5jZVtpXSkpIHsKLSAgICAg ICAgICAgICAgICByZXN1bHQgPSAiRkFJTCgnIitwcm9wW2ldKyInIHdhczoiK3RbcHJvcFtpXV0r IiwgZXhwZWN0ZWQ6IitleHBlY3RlZFt3aGljaF1baV0rIikiOwotICAgICAgICAgICAgICAgIHJl dHVybjsKLSAgICAgICAgICAgIH0KKyAgICAgICAgZm9yIChpID0gMDsgaSA8IDY7ICsraSkgewor ICAgICAgICAgICAgaWYgKCFpc0VxdWFsKHRbcHJvcFtpXV0sIGV4cGVjdGVkW3doaWNoXVtpXSwg dG9sZXJhbmNlW2ldKSkKKyAgICAgICAgICAgICAgICByZXN1bHQgKz0gIkZBSUwoJyIrcHJvcFtp XSsiJyB3YXM6Iit0W3Byb3BbaV1dKyIsIGV4cGVjdGVkOiIrZXhwZWN0ZWRbd2hpY2hdW2ldKyIp IjsKKyAgICAgICAgfQorICAgICAgICAKKyAgICAgICAgbnVtU25hcHNob3RzKys7CisgICAgICAg IGlmIChudW1TbmFwc2hvdHMgPT0gMikgeworICAgICAgICAgICAgZG9jdW1lbnQuZ2V0RWxlbWVu dEJ5SWQoJ3Jlc3VsdCcpLmlubmVySFRNTCA9IHJlc3VsdCB8fCAiUEFTUyI7CisgICAgICAgICAg ICBpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgICAgICAgICAgICAgIGxheW91 dFRlc3RDb250cm9sbGVyLm5vdGlmeURvbmUoKTsKKyAgICAgICAgfQogICAgIH0KIAogICAgIGZ1 bmN0aW9uIHN0YXJ0KCkKICAgICB7Ci0gICAgICAgIHNldFRpbWVvdXQoInNuYXBzaG90KDApIiwg NTAwKTsKLSAgICAgICAgc2V0VGltZW91dCgic25hcHNob3QoMSkiLCAxMDAwKTsKLSAgICAgICAg Ci0gICAgICAgIHdpbmRvdy5zZXRUaW1lb3V0KGZ1bmN0aW9uKCkgewotICAgICAgICAgICAgZG9j dW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ3Jlc3VsdCcpLmlubmVySFRNTCA9IHJlc3VsdDsKLSAgICAg ICAgICAgIGlmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCi0gICAgICAgICAgICAgICAg bGF5b3V0VGVzdENvbnRyb2xsZXIubm90aWZ5RG9uZSgpOwotICAgICAgICB9LCAxMTAwKTsKKyAg ICAgICAgc2V0VGltZW91dCgic25hcHNob3QoMCkiLCAyNTApOworICAgICAgICBzZXRUaW1lb3V0 KCJzbmFwc2hvdCgxKSIsIDUwMCk7CiAgICAgfQogICAgIAogICAgIGRvY3VtZW50LmFkZEV2ZW50 TGlzdGVuZXIoJ3dlYmtpdEFuaW1hdGlvblN0YXJ0Jywgc3RhcnQsIGZhbHNlKTsKQEAgLTY4LDgg KzY3LDggQEAKICAgPC9zY3JpcHQ+CiA8L2hlYWQ+CiA8Ym9keT4KLVRoaXMgdGVzdCBwZXJmb3Jt cyBhbiBhbmltYXRpb24gb2YgdGhlIHRyYW5zbGF0ZSgpIGFuZCBzY2FsZSgpIG9wZXJhdG9ycy4g SXQgYW5pbWF0ZXMgb3ZlciAyIHNlY29uZHMuIAotQXQgMC41IGFuZCAxIHNlY29uZCBpdCB0YWtl cyBhIHNuYXBzaG90IGFuZCBleHBlY3RzIHRoZSByZXN1bHQgdG8gYmUgd2l0aGluIHJhbmdlLgor VGhpcyB0ZXN0IHBlcmZvcm1zIGFuIGFuaW1hdGlvbiBvZiB0aGUgdHJhbnNsYXRlKCkgYW5kIHNj YWxlKCkgb3BlcmF0b3JzLiBJdCBhbmltYXRlcyBvdmVyIDEgc2Vjb25kcy4gCitBdCAwLjI1IGFu ZCAwLjUgc2Vjb25kcyBpdCB0YWtlcyBhIHNuYXBzaG90IGFuZCBleHBlY3RzIHRoZSByZXN1bHQg dG8gYmUgd2l0aGluIHJhbmdlLgogPGRpdiBpZD0iYm94Ij4KIDwvZGl2PgogPGRpdiBpZD0icmVz dWx0Ij4K