40537 2010-06-12 21:21:04 -0700 REGRESSION (r60974): Continuous wheel scrolling is too fast in list boxes and in Safari Reader 2010-06-13 10:22:22 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Mac OS X 10.6 RESOLVED FIXED InRadar, Regression P1 Normal --- 1 mitz webkit-unassigned abarth eric hyatt webkit.review.bot oldest_to_newest 237469 0 mitz 2010-06-12 21:21:04 -0700 Using Magic Mouse to scroll a list boxes (such as the Component list at <https://bugs.webkit.org/enter_bug.cgi?product=WebKit>) and Safari Reader, scrolling is way faster in TOT than in Safari 5. I think this is a regression from <http://trac.webkit.org/changeset/60974> and <http://trac.webkit.org/changeset/60978>. 237470 1 mitz 2010-06-12 21:21:49 -0700 <rdar://problem/8087385> 237512 2 hyatt 2010-06-13 00:19:51 -0700 Did I botch the rollout or did new code get added that depended on the changed behavior 237567 3 mitz 2010-06-13 09:13:50 -0700 (In reply to comment #2) > Did I botch the rollout or did new code get added that depended on the changed behavior Looks like the former: <http://trac.webkit.org/changeset?old_path=%2Ftrunk%2FWebCore%2Fplatform%2Fmac%2FWheelEventMac.mm&old=56011&new_path=%2Ftrunk%2FWebCore%2Fplatform%2Fmac%2FWheelEventMac.mm&new=60974>. 237571 4 58594 mitz 2010-06-13 09:26:14 -0700 Created attachment 58594 Don’t multiply continuous deltas by pixelsPerLineStep() 237573 5 mitz 2010-06-13 09:41:13 -0700 Fixed in <http://trac.webkit.org/changeset/61092>. 237576 6 webkit.review.bot 2010-06-13 10:17:48 -0700 http://trac.webkit.org/changeset/61092 might have broken SnowLeopard Intel Release (Tests) 237577 7 mitz 2010-06-13 10:22:22 -0700 Updated test results in <http://trac.webkit.org/changeset/61093>. 58594 2010-06-13 09:26:14 -0700 2010-06-13 09:38:23 -0700 Don’t multiply continuous deltas by pixelsPerLineStep() 40537_r1.diff text/plain 1440 mitz SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA2MTA5MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTQgQEAKKzIwMTAtMDYtMTMgIERhbiBCZXJuc3RlaW4gIDxtaXR6QGFwcGxlLmNv bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICA8cmRh cjovL3Byb2JsZW0vODA4NzM4NT4gUkVHUkVTU0lPTiAocjYwOTc0KTogQ29udGludW91cyB3aGVl bCBzY3JvbGxpbmcgaXMgdG9vIGZhc3QgaW4gbGlzdCBib3hlcyBhbmQgaW4gU2FmYXJpIFJlYWRl cgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDA1MzcK KworICAgICAgICAqIHBsYXRmb3JtL21hYy9XaGVlbEV2ZW50TWFjLm1tOgorICAgICAgICAoV2Vi Q29yZTo6UGxhdGZvcm1XaGVlbEV2ZW50OjpQbGF0Zm9ybVdoZWVsRXZlbnQpOiBPbmx5IG11bHRp cGx5IGJ5IHBpeGVsc1BlckxpbmVTdGVwKCkgaWYgdGhlCisgICAgICAgIGV2ZW50IGlzIG5vdCBj b250aW51b3VzLCBhcyB3YXMgYmVmb3JlIHI1NjAxMi4KKwogMjAxMC0wNi0xMyAgQW5kcmFzIEJl Y3NpICA8YWJlY3NpQHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFyaW4gQWRs ZXIuCkluZGV4OiBXZWJDb3JlL3BsYXRmb3JtL21hYy9XaGVlbEV2ZW50TWFjLm1tCj09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT0KLS0tIFdlYkNvcmUvcGxhdGZvcm0vbWFjL1doZWVsRXZlbnRNYWMubW0JKHJldmlzaW9uIDYx MDYzKQorKysgV2ViQ29yZS9wbGF0Zm9ybS9tYWMvV2hlZWxFdmVudE1hYy5tbQkod29ya2luZyBj b3B5KQpAQCAtNTEsMTAgKzUxLDkgQEAgUGxhdGZvcm1XaGVlbEV2ZW50OjpQbGF0Zm9ybVdoZWVs RXZlbnQoTgogICAgIH0gZWxzZSB7CiAgICAgICAgIG1fd2hlZWxUaWNrc1ggPSBtX2RlbHRhWDsK ICAgICAgICAgbV93aGVlbFRpY2tzWSA9IG1fZGVsdGFZOworICAgICAgICBtX2RlbHRhWCAqPSBz dGF0aWNfY2FzdDxmbG9hdD4oU2Nyb2xsYmFyOjpwaXhlbHNQZXJMaW5lU3RlcCgpKTsKKyAgICAg ICAgbV9kZWx0YVkgKj0gc3RhdGljX2Nhc3Q8ZmxvYXQ+KFNjcm9sbGJhcjo6cGl4ZWxzUGVyTGlu ZVN0ZXAoKSk7CiAgICAgfQotCi0gICAgbV9kZWx0YVggKj0gc3RhdGljX2Nhc3Q8ZmxvYXQ+KFNj cm9sbGJhcjo6cGl4ZWxzUGVyTGluZVN0ZXAoKSk7Ci0gICAgbV9kZWx0YVkgKj0gc3RhdGljX2Nh c3Q8ZmxvYXQ+KFNjcm9sbGJhcjo6cGl4ZWxzUGVyTGluZVN0ZXAoKSk7CiB9CiAKIH0gLy8gbmFt ZXNwYWNlIFdlYkNvcmUK