81519 2012-03-19 09:32:40 -0700 TiledBackingStore: When checking if the visible area is covered, only convert to scaled coordinates once. 2012-03-22 11:34:43 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 79666 79668 1 jturcotte jturcotte kenneth ossy szledan oldest_to_newest 581868 0 jturcotte 2012-03-19 09:32:40 -0700 TiledBackingStore: When checking if the visible area is covered, only convert to scaled coordinates once. 581873 1 132599 jturcotte 2012-03-19 09:36:13 -0700 Created attachment 132599 Patch 584276 2 jturcotte 2012-03-21 10:17:15 -0700 Committed r111560: <http://trac.webkit.org/changeset/111560> 585328 3 szledan 2012-03-22 08:36:35 -0700 This test: compositing/geometry/preserve-3d-switching.html fails(timeout) on x86-32 Linux Qt version. After this patch these tests keep crashing: compositing/geometry/layer-due-to-layer-children.html compositing/geometry/repaint-foreground-layer.html. So this bug should be reopened. --- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/compositing/geometry/preserve-3d-switching-expected.txt +++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/compositing/geometry/preserve-3d-switching-actual.txt @@ -1,39 +1,1 @@ -The green box appear angled out from the yellow box and embedded in it. - --webkit-transform: translateZ(-100px) rotateY(45deg); -(GraphicsLayer - (bounds 800.00 600.00) - (children 1 - (GraphicsLayer - (bounds 800.00 600.00) - (children 1 - (GraphicsLayer - (position 108.00 74.00) - (bounds 304.00 304.00) - (drawsContent 1) - (childrenTransform [1.00 0.00 0.00 0.00] [0.00 1.00 0.00 0.00] [0.00 0.00 1.00 -0.00] [0.00 0.00 0.00 1.00]) - (children 1 - (GraphicsLayer - (position 12.00 12.00) - (bounds 280.00 280.00) - (opacity 0.80) - (preserves3D 1) - (drawsContent 1) - (transform [0.77 -0.56 -0.32 0.00] [0.00 0.50 -0.87 0.00] [0.64 0.66 0.38 0.00] [0.00 0.00 0.00 1.00]) - (children 1 - (GraphicsLayer - (position 40.00 40.00) - (bounds 200.00 200.00) - (opacity 0.70) - (drawsContent 1) - (transform [0.77 0.00 0.64 0.00] [0.00 1.00 0.00 0.00] [-0.64 0.00 0.77 0.00] [0.00 0.00 50.00 1.00]) - ) - ) - ) - ) - ) - ) - ) - ) -) - +Timed out waiting for final message from web process 585332 4 ossy 2012-03-22 08:41:35 -0700 Reopen based on Comment #3 585386 5 szledan 2012-03-22 09:41:46 -0700 Skipped on Qt WK2 - http://trac.webkit.org/changeset/111711/trunk/LayoutTests/platform/qt-5.0-wk2/Skipped Please unskip with the proper fix. 585523 6 jturcotte 2012-03-22 11:33:24 -0700 Committed r111735: <http://trac.webkit.org/changeset/111735> 585524 7 jturcotte 2012-03-22 11:34:43 -0700 Should be fixed now, thanks for finding it out. I couldn't reproduce the crash on the two other tests but they were probably caused by the same problem so I unskipped them too. 132599 2012-03-19 09:36:13 -0700 2012-03-22 08:41:48 -0700 Patch bug-81519-20120319173616.patch text/plain 1627 jturcotte U3VidmVyc2lvbiBSZXZpc2lvbjogMTEwODQ5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNjQ3Njc1M2NjODRkZjgz MDgzYWVlNDI2ZjY2ODllN2I3NDNmYjMyNS4uZjY2NzFkMzY3ZGVmN2E2MjRhMGNmZDhmOWNmYzlm ZGYyY2EyYjdjNSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEyLTAzLTE5ICBKb2Nl bHluIFR1cmNvdHRlICA8am9jZWx5bi50dXJjb3R0ZUBub2tpYS5jb20+CisKKyAgICAgICAgVGls ZWRCYWNraW5nU3RvcmU6IFdoZW4gY2hlY2tpbmcgaWYgdGhlIHZpc2libGUgYXJlYSBpcyBjb3Zl cmVkLCBvbmx5IGNvbnZlcnQgdG8gc2NhbGVkIGNvb3JkaW5hdGVzIG9uY2UuCisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTUxOQorCisgICAgICAgIFJl dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEJvdGggdmlzaWJsZVJlY3QgYW5k IGNvdmVyYWdlUmF0aW8gY2FsbCBtYXBGcm9tQ29udGVudHMgb24gdGhlIGlucHV0IHJlY3QsCisg ICAgICAgIHRoaXMgd291bGMgY2F1c2UgdGhlIGNvdmVyYWdlUmF0aW8gdG8gbmV2ZXIgcmVhY2gg MS4wIHdoZW4gdGhlIGNvbnRlbnRzIGlzCisgICAgICAgIHNjYWxlZCB1cC4KKworICAgICAgICAq IHBsYXRmb3JtL2dyYXBoaWNzL1RpbGVkQmFja2luZ1N0b3JlLmNwcDoKKyAgICAgICAgKFdlYkNv cmU6OlRpbGVkQmFja2luZ1N0b3JlOjp2aXNpYmxlQXJlYUlzQ292ZXJlZCk6CisKIDIwMTItMDMt MTUgIE5pa29sYXMgWmltbWVybWFubiAgPG56aW1tZXJtYW5uQHJpbS5jb20+CiAKICAgICAgICAg RW5hYmxlIGFuaW1WYWwgc3VwcG9ydCBmb3IgU1ZHTnVtYmVyTGlzdApkaWZmIC0tZ2l0IGEvU291 cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvVGlsZWRCYWNraW5nU3RvcmUuY3BwIGIvU291 cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvVGlsZWRCYWNraW5nU3RvcmUuY3BwCmluZGV4 IDliZDQyNGM5NDE3ZWNhZGZlOGJiZmIzMmYxMDgwZTIzODJiOGJhYTUuLjljNzNkNGRlNmU1NGNl YzRlNDM1NTlmYWY1NGZiYTQwMTA4ODE2ZTEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3Bs YXRmb3JtL2dyYXBoaWNzL1RpbGVkQmFja2luZ1N0b3JlLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29y ZS9wbGF0Zm9ybS9ncmFwaGljcy9UaWxlZEJhY2tpbmdTdG9yZS5jcHAKQEAgLTIyMyw3ICsyMjMs NyBAQCBmbG9hdCBUaWxlZEJhY2tpbmdTdG9yZTo6Y292ZXJhZ2VSYXRpbyhjb25zdCBXZWJDb3Jl OjpJbnRSZWN0JiBjb250ZW50c1JlY3QpIGNvbgogCiBib29sIFRpbGVkQmFja2luZ1N0b3JlOjp2 aXNpYmxlQXJlYUlzQ292ZXJlZCgpIGNvbnN0CiB7Ci0gICAgcmV0dXJuIGNvdmVyYWdlUmF0aW8o dmlzaWJsZVJlY3QoKSkgPT0gMS4wZjsKKyAgICByZXR1cm4gY292ZXJhZ2VSYXRpbyhtX2NsaWVu dC0+dGlsZWRCYWNraW5nU3RvcmVWaXNpYmxlUmVjdCgpKSA9PSAxLjBmOwogfQogCiB2b2lkIFRp bGVkQmFja2luZ1N0b3JlOjpjcmVhdGVUaWxlcygpCg==