52845 2011-01-20 14:11:34 -0800 REGRESSION (r75138-r75503): Animations on Apple HTML5 Gallery demo are wrong 2011-03-14 13:54:29 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) PC Windows XP RESOLVED FIXED http://www.apple.com/html5/showcase/gallery/ InRadar, PlatformOnly P2 Normal --- 1 aroben cmarrin cmarrin simon.fraser oldest_to_newest 337595 0 aroben 2011-01-20 14:11:34 -0800 To reproduce: 1. Go to http://www.apple.com/html5/showcase/gallery/ 2. Click on "Grid" 3. Hover over some images The images are supposed to animate to be slightly larger. Instead they pop into the wrong location without any animation. All of the other gallery styles have similar problems. The 3D gallery styles are particularly obviously wrong. 337637 1 aroben 2011-01-20 15:09:24 -0800 I don't have any builds available between r75475 and r75503. Oddly, the only change in that range that seems remotely related is r75494, where I deleted WKCACFLayer.cpp/h. But those files weren't being used! 337641 2 aroben 2011-01-20 15:13:42 -0800 <rdar://problem/8894958> 338014 3 aroben 2011-01-21 05:18:11 -0800 Oh, wait. Some of these builds are in the revision range where we changed the layout of the WebKitBuild directory, which means that the revision numbers aren't accurate (the bots were building into one directory and archiving build products from another). This means that my regression range is really larger than I thought. 338016 4 aroben 2011-01-21 05:25:29 -0800 I'm highly suspicious of r75262. 338159 5 cmarrin 2011-01-21 10:51:26 -0800 (In reply to comment #4) > I'm highly suspicious of r75262. Yeah, this is a good bet. It is where we went from software to hardware animation for compositing layers. 367168 6 cmarrin 2011-03-14 13:20:47 -0700 *** Bug 53495 has been marked as a duplicate of this bug. *** 367169 7 85709 cmarrin 2011-03-14 13:22:41 -0700 Created attachment 85709 Patch 367172 8 85709 aroben 2011-03-14 13:25:27 -0700 Comment on attachment 85709 Patch Is there a way to make a regression test for this? 367197 9 cmarrin 2011-03-14 13:54:29 -0700 Committed r81054: <http://trac.webkit.org/changeset/81054> 85709 2011-03-14 13:22:41 -0700 2011-03-14 13:25:27 -0700 Patch bug-52845-20110314132200.patch text/plain 2397 cmarrin SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDgxMDQ2KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMjAgQEAKKzIwMTEtMDMtMTQgIENocmlzIE1h cnJpbiAgPGNtYXJyaW5AYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo T09QUyEpLgorCisgICAgICAgIFJFR1JFU1NJT04gKHI3NTEzOC1yNzU1MDMpOiBBbmltYXRpb25z IG9uIEFwcGxlIEhUTUw1IEdhbGxlcnkgZGVtbyBhcmUgd3JvbmcKKyAgICAgICAgaHR0cHM6Ly9i dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTUyODQ1CisKKyAgICAgICAgVGhlIGxhc3RD b21taXRUaW1lKCkgdmFsdWUgaW4gQ0FDRkxheWVyVHJlZUhvc3Qgd2FzIHJldHVybmluZyBhcyB0 aGUgdGltZQorICAgICAgICB0aGUgcmVuZGVyIHByZXZpb3VzIHRvIHRoaXMgb25lIGhhcHBlbmVk LiBUaGF0IG9mdGVuIG1hZGUgaXQgc2VlbSBsaWtlCisgICAgICAgIGFuaW1hdGlvbnMgc3RhcnRl ZCBtb3JlIGluIHRoZSBwYXN0IHRoYW4gdGhleSBkaWQsIGJyZWFraW5nIG1hbnkgYW5pbWF0aW9u cy4KKyAgICAgICAgVGhlIHN0YXJ0QW5pbWF0aW9ucygpIGNhbGwgYWN0dWFsbHkgZmlyZXMgZnJv bSBhIENBQ0YgY2FsbGJhY2sgYWZ0ZXIgYWxsIHRoZQorICAgICAgICBXZWJLaXQgY29udGVudCBo YXMgYmVlbiByZW5kZXJlZC4gU28gc2VuZGluZyBjdXJyZW50VGltZSBhcyB0aGUgc3RhcnQgdGlt ZQorICAgICAgICB0byB0aGUgYW5pbWF0aW9ucyBpcyBjbG9zZSBlbm91Z2ggZm9yIHByb3BlciBz eW5jaHJvbml6YXRpb24uCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9jYS93aW4vQ0FD RkxheWVyVHJlZUhvc3QuY3BwOgorICAgICAgICAoV2ViQ29yZTo6Q0FDRkxheWVyVHJlZUhvc3Q6 Om5vdGlmeUFuaW1hdGlvbnNTdGFydGVkKToKKwogMjAxMS0wMy0xMSAgRGF2aWQgSHlhdHQgIDxo eWF0dEBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgU2ltb24gRnJhc2VyLgpJbmRl eDogU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvY2Evd2luL0NBQ0ZMYXllclRyZWVI b3N0LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9j YS93aW4vQ0FDRkxheWVyVHJlZUhvc3QuY3BwCShyZXZpc2lvbiA4MDg4NSkKKysrIFNvdXJjZS9X ZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2NhL3dpbi9DQUNGTGF5ZXJUcmVlSG9zdC5jcHAJKHdv cmtpbmcgY29weSkKQEAgLTMwNywxNyArMzA3LDE0IEBAIHZvaWQgQ0FDRkxheWVyVHJlZUhvc3Q6 OmNvbnRleHREaWRDaGFuZ2UKIAogdm9pZCBDQUNGTGF5ZXJUcmVlSG9zdDo6bm90aWZ5QW5pbWF0 aW9uc1N0YXJ0ZWQoKQogeworICAgIC8vIFNlbmQgY3VycmVudFRpbWUgdG8gdGhlIHBlbmRpbmcg YW5pbWF0aW9ucy4gVGhpcyBmdW5jdGlvbiBpcyBjYWxsZWQgYnkgQ0FDRiBpbiBhIGNhbGxiYWNr CisgICAgLy8gd2hpY2ggb2NjdXJzIGFmdGVyIHRoZSBkcmF3SW5Db250ZXh0IGNhbGxzLiBTbyBj dXJyZW50VGltZSBpcyB2ZXJ5IGNsb3NlIHRvIHRoZSB0aW1lCisgICAgLy8gdGhlIGFuaW1hdGlv bnMgYWN0dWFsbHkgc3RhcnQKICAgICBkb3VibGUgY3VycmVudFRpbWUgPSBXVEY6OmN1cnJlbnRU aW1lKCk7Ci0gICAgZG91YmxlIGNvbW1pdFRpbWUgPSBsYXN0Q29tbWl0VGltZSgpOwotICAgIGRv dWJsZSBtZWRpYVRpbWUgPSBDQUN1cnJlbnRNZWRpYVRpbWUoKTsKLSAgICBpZiAoY29tbWl0VGlt ZSA8PSAwKQotICAgICAgICBjb21taXRUaW1lID0gbWVkaWFUaW1lOwotICAgIGRvdWJsZSB0aW1l ID0gY3VycmVudFRpbWUgKyBjb21taXRUaW1lIC0gbWVkaWFUaW1lOwotICAgIEFTU0VSVCh0aW1l IDw9IGN1cnJlbnRUaW1lKTsKIAogICAgIEhhc2hTZXQ8UmVmUHRyPFBsYXRmb3JtQ0FMYXllcj4g Pjo6aXRlcmF0b3IgZW5kID0gbV9wZW5kaW5nQW5pbWF0ZWRMYXllcnMuZW5kKCk7CiAgICAgZm9y IChIYXNoU2V0PFJlZlB0cjxQbGF0Zm9ybUNBTGF5ZXI+ID46Oml0ZXJhdG9yIGl0ID0gbV9wZW5k aW5nQW5pbWF0ZWRMYXllcnMuYmVnaW4oKTsgaXQgIT0gZW5kOyArK2l0KQotICAgICAgICAoKml0 KS0+YW5pbWF0aW9uU3RhcnRlZCh0aW1lKTsKKyAgICAgICAgKCppdCktPmFuaW1hdGlvblN0YXJ0 ZWQoY3VycmVudFRpbWUpOwogCiAgICAgbV9wZW5kaW5nQW5pbWF0ZWRMYXllcnMuY2xlYXIoKTsK IH0K