43905 2010-08-12 04:39:06 -0700 [Qt] animations tests fails which uses suspend/resume 2012-01-20 06:54:19 -0800 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) All All RESOLVED FIXED Qt, QtTriaged P2 Normal --- 48161 49182 1 ossy menard cmarrin eric joybro201 menard oliver ossy oldest_to_newest 263712 0 ossy 2010-08-12 04:39:06 -0700 animations/suspend-resume-animation.html introduced in http://trac.webkit.org/changeset/65107 and it fails on the Qt bot: --- /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/animations/suspend-resume-animation-expected.txt 2010-08-12 04:13:57.692174443 -0700 +++ /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/animations/suspend-resume-animation-actual.txt 2010-08-12 04:13:57.692174443 -0700 @@ -2,5 +2,5 @@ PASS - "webkitTransform" property for "box" element at 0.6s saw something close to: 1,0,0,1,200,0 PASS - "webkitTransform" property for "box" element at 0.9s saw something close to: 1,0,0,1,200,0 -PASS - "webkitTransform" property for "box" element at 1.5s saw something close to: 1,0,0,1,400,0 +FAIL - "webkitTransform" property for "box" element at 1.5s expected: 1,0,0,1,400,0 but saw: matrix(1, 0, 0, 1, 212, 0) 263726 1 ossy 2010-08-12 05:08:10 -0700 Skipped by http://trac.webkit.org/changeset/65236 until fix. 305569 2 cmarrin 2010-11-05 15:12:26 -0700 From the failure, it looks like suspend is working, but resume is not. A new test is failing which also uses suspend/resume: LayoutTests/animations/stop-animation-on-suspend.html 305588 3 ossy 2010-11-05 15:35:07 -0700 (In reply to comment #2) > From the failure, it looks like suspend is working, but resume is not. > > A new test is failing which also uses suspend/resume: > > LayoutTests/animations/stop-animation-on-suspend.html stop-animation-on-suspend.html introduced in http://trac.webkit.org/changeset/71424 pretty diffs: http://build.webkit.org/results/Qt%20Linux%20Release/r71452%20(23266)/animations/stop-animation-on-suspend-pretty-diff.html http://build.webkit.org/results/Qt%20Linux%20Release/r71452%20(23266)/animations/suspend-resume-animation-events-pretty-diff.html I skipped them: http://trac.webkit.org/changeset/71453 415618 4 joybro201 2011-06-05 10:06:31 -0700 (In reply to comment #3) > (In reply to comment #2) > > From the failure, it looks like suspend is working, but resume is not. > > > > A new test is failing which also uses suspend/resume: > > > > LayoutTests/animations/stop-animation-on-suspend.html > > stop-animation-on-suspend.html introduced in http://trac.webkit.org/changeset/71424 > > pretty diffs: > http://build.webkit.org/results/Qt%20Linux%20Release/r71452%20(23266)/animations/stop-animation-on-suspend-pretty-diff.html > http://build.webkit.org/results/Qt%20Linux%20Release/r71452%20(23266)/animations/suspend-resume-animation-events-pretty-diff.html > > I skipped them: http://trac.webkit.org/changeset/71453 animations/suspend-resume-animation.html animations/suspend-resume-animation-events.html animations/stop-animation-on-suspend.html In r88132, webkit-qt is passing the above tests you skipped. Don't these tests have to be unskipped? :) 539462 5 123305 menard 2012-01-20 06:36:44 -0800 Created attachment 123305 Patch 539470 6 123305 ossy 2012-01-20 06:53:53 -0800 Comment on attachment 123305 Patch r=me and landed in http://trac.webkit.org/changeset/105511 123305 2012-01-20 06:36:44 -0800 2012-01-20 06:54:19 -0800 Patch bug-43905-20120120113634.patch text/plain 2124 menard U3VidmVyc2lvbiBSZXZpc2lvbjogMTA1NTA0CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggMmExM2JmMjgxOTE0NzRhZjkwYzcx NjIxZDZhYmU0MTQ5NjgwMDVkNS4uZDUwNjNmNWUwNTA4NzI4OGIyNTM0Y2IxMDE5ZWU0ZjMwZmI4 N2RmNiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE2IEBACiAyMDEyLTAxLTIwICBBbGV4aXMgTWVuYXJkICA8 YWxleGlzLm1lbmFyZEBvcGVuYm9zc2Eub3JnPgogCisgICAgICAgIFtRdF0gYW5pbWF0aW9ucyB0 ZXN0cyBmYWlscyB3aGljaCB1c2VzIHN1c3BlbmQvcmVzdW1lCisgICAgICAgIGh0dHBzOi8vYnVn cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00MzkwNQorCisgICAgICAgIFJldmlld2VkIGJ5 IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoZXkgc2VlbSB0byBwYXNzIGNvcnJlY3RseSBv biBteSBtYWNoaW5lLgorCisgICAgICAgICogcGxhdGZvcm0vcXQvU2tpcHBlZDoKKworMjAxMi0w MS0yMCAgQWxleGlzIE1lbmFyZCAgPGFsZXhpcy5tZW5hcmRAb3BlbmJvc3NhLm9yZz4KKwogICAg ICAgICByZW1vdmUgQ1NTQm9yZGVySW1hZ2VWYWx1ZQogICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzU1NjMKIApkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMv cGxhdGZvcm0vcXQvU2tpcHBlZCBiL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0L1NraXBwZWQKaW5k ZXggOWEzMmYyODEyNzRkMjFhZWM1MzUxYTcwMWNhZDVmYzFkMjA0ODg3My4uNzhlMGUzNzRjOThh M2EwYTIyZTcxNTZlZGY3NmY1M2I3ZGM0YjdlZSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxh dGZvcm0vcXQvU2tpcHBlZAorKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9Ta2lwcGVkCkBA IC04NjksMTMgKzg2OSw2IEBAIHN2Zy90ZXh0L3RleHQtaW50cm8tMDUtdC5zdmcKIHN2Zy90ZXh0 L3RleHQtdHNlbGVjdC0wMi1mLnN2ZwogCiAjID09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ICMKLSMgZmFp bGluZyBhbmltYXRpb25zIHRlc3RzCi0jID09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ICMKLSMgTmVlZHMg ZXhwZWN0ZWQgcmVzdWx0cwotYW5pbWF0aW9ucy9hbmltYXRpb24tZHJ0LWFwaS5odG1sCi1hbmlt YXRpb25zL2FuaW1hdGlvbi1kcnQtYXBpLW11bHRpcGxlLWtleWZyYW1lcy5odG1sCi0KLSMgPT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT0gIwogIyBGYWlsaW5nIENTUyBUZXN0cwogIyA9PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PSAjCiAjIFJFR1JFU1NJT04gKHI1MzQ1MCkgLSBza2lwcGVkIHVudGlsIGZpeApAQCAt MTczMiwxMiArMTcyNSw2IEBAIGZhc3QvZm9ybXMvdmFsaWRhdGlvbi1tZXNzYWdlLW9uLXRleHRh cmVhLmh0bWwKICMgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTQyODIx CiBhbmltYXRpb25zL3BsYXktc3RhdGUuaHRtbAogCi0jIFtRdF0gYW5pbWF0aW9ucyB0ZXN0cyBm YWlscyB3aGljaCB1c2VzIHN1c3BlbmQvcmVzdW1lCi0jIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD00MzkwNQotYW5pbWF0aW9ucy9zdXNwZW5kLXJlc3VtZS1hbmltYXRp b24uaHRtbAotYW5pbWF0aW9ucy9zdXNwZW5kLXJlc3VtZS1hbmltYXRpb24tZXZlbnRzLmh0bWwK LWFuaW1hdGlvbnMvc3RvcC1hbmltYXRpb24tb24tc3VzcGVuZC5odG1sCi0KICMgQXNzdW1lcyBz cGVzaWZpYyBtZXRyaWNzCiAjIFNlZSBodHRwOi8vY29kZS5nb29nbGUuY29tL3AvY2hyb21pdW0v aXNzdWVzL2RldGFpbD9pZD0xOTg5Nwogc3ZnL2N1c3RvbS9nZXRzY3JlZW5jdG0taW4tbWl4ZWQt Y29udGVudDIueGh0bWwK