38416 2010-04-30 18:24:15 -0700 fast/repaint/fixed-move-after-keyboard-scroll passes on Leopard, fails on Snow Leopard 2010-05-11 19:07:52 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Other OS X 10.5 RESOLVED FIXED P2 Normal --- 38964 1 jamesr webkit-unassigned abarth eric simon.fraser oldest_to_newest 219677 0 jamesr 2010-04-30 18:24:15 -0700 fast/repaint/fixed-move-after-keyboard-scroll passes on Leopard, fails on Snow Leopard 219678 1 jamesr 2010-04-30 18:28:50 -0700 This is a new test. It passes on Leopard and most other platforms WebKit runs on, but seems to fail on Snow Leopard. Based on the test output I suspect that the test is finishing before the test body runs. I've verified that the test passes when run manually. The test depends on eventSender.keyDown("pageDown") dispatching a synchronous scroll event. This is true on Leopard, but might not be true on Snow Leopard (perhaps due to fancy scroll animations or some such?). If this really is what is happening, it should be pretty easy to modify the test to work around this. 219679 2 jamesr 2010-04-30 18:30:14 -0700 I won't get a chance to look into this until next week, so I would like to add it to the SL skipped list until then. 219680 3 54843 jamesr 2010-04-30 18:33:09 -0700 Created attachment 54843 Patch 219771 4 54843 eric 2010-05-01 21:53:54 -0700 Comment on attachment 54843 Patch You should have just checked this in. it's impossible to CQ this since the tree is currently on fire because of this. Broken tree = very lame. 219774 5 54843 eric 2010-05-01 22:07:35 -0700 Comment on attachment 54843 Patch Clearing flags on attachment: 54843 Committed r58639: <http://trac.webkit.org/changeset/58639> 219775 6 eric 2010-05-01 22:07:49 -0700 All reviewed patches have been landed. Closing bug. 219776 7 eric 2010-05-01 22:09:02 -0700 I'm very sad. The tree was needlessly red for over a day due to the original change. Total lame. Fixed now, I believe. 219875 8 simon.fraser 2010-05-02 09:45:03 -0700 Sorry, I promised James I'd look at this bug other stuff came up. 224242 9 jamesr 2010-05-11 16:38:18 -0700 I now have access to a Snow Leopard machine (hurray!) am investigating this. 224323 10 jamesr 2010-05-11 18:52:34 -0700 This seems to boil down to smooth scrolling being on by default in Snow Leopard and off by default in Leopard. With "Use smooth scrolling" disabled in my System Preferences on Snow Leopard the test passes unmodified. The options are: - teach DRT how to turn smooth scrolling off when running layout tests - teach this test to detect when smooth scrolling is going on and accomodate it I think the first option is better. 54843 2010-04-30 18:33:09 -0700 2010-05-01 22:07:35 -0700 Patch bug-38416-20100430183307.patch text/plain 1479 jamesr SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDU4NjI2KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTAtMDQtMzAgIEphbWVzIFJvYmluc29uICA8 amFtZXNyQGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMh KS4KKworICAgICAgICBBZGQgZmFzdC9yZXBhaW50L2ZpeGVkLW1vdmUtYWZ0ZXIta2V5Ym9hcmQt c2Nyb2xsLmh0bWwgdG8gc2tpcHBlZCBsaXN0IHBlbmRpbmcgaW52ZXN0aWdhdGlvbgorICAgICAg ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mzg0MTYKKworICAgICAg ICAqIHBsYXRmb3JtL21hYy1zbm93bGVvcGFyZC9Ta2lwcGVkOgorCiAyMDEwLTA0LTMwICBDaHJp cyBNYXJyaW4gIDxjbWFycmluQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBTaW1v biBGcmFzZXIuCkluZGV4OiBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtc25vd2xlb3BhcmQvU2tp cHBlZAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtc25vd2xlb3BhcmQv U2tpcHBlZAkocmV2aXNpb24gNTg2MjYpCisrKyBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtc25v d2xlb3BhcmQvU2tpcHBlZAkod29ya2luZyBjb3B5KQpAQCAtMTI1LDMgKzEyNSw3IEBAIHRyYW5z aXRpb25zL3RyYW5zaXRpb24tZW5kLWV2ZW50LWRlc3Ryb3kKICMgaHR0cHM6Ly9idWdzLndlYmtp dC5vcmcvc2hvd19idWcuY2dpP2lkPTM4MzU2CiBmYXN0L2pzL3NwdXRuaWsvQ29uZm9ybWFuY2Uv MTFfRXhwcmVzc2lvbnMvMTEuNV9NdWx0aXBsaWNhdGl2ZV9PcGVyYXRvcnMvMTEuNS4zX0FwcGx5 aW5nX3RoZV9wZXJjZW50X09wZXJhdG9yL1MxMS41LjNfQTRfVDIuaHRtbAogZmFzdC9qcy9zcHV0 bmlrL0NvbmZvcm1hbmNlLzE1X05hdGl2ZV9PYmplY3RzLzE1LjRfQXJyYXkvMTUuNC40LzE1LjQu NC4xMl9BcnJheV9wcm90b3R5cGVfc3BsaWNlL1MxNS40LjQuMTJfQTIuMV9UMy5odG1sCisKKyMg VGVzdCBmYWlscyBvbiBTbm93IExlb3BhcmQgaW4gRFJUIGJ1dCBwYXNzZXMgd2hlbiBydW4gbWFu dWFsbHkgYW5kIHBhc3NlcyBvbiBMZW9wYXJkLiBUaGUgdGVzdCBpdHNlbGYgbWlnaHQgbmVlZAor IyB0d2Vha2luZyB0byBkZWFsIHdpdGggc2Nyb2xsaW5nIGRpZmZlcmVuY2VzIGJldHdlZW4gTGVv cGFyZCBhbmQgU0wuIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zODQx NgorZmFzdC9yZXBhaW50L2ZpeGVkLW1vdmUtYWZ0ZXIta2V5Ym9hcmQtc2Nyb2xsLmh0bWwK