23819 2009-02-07 08:06:13 -0800 REGRESSION (r36869): Select scroller is shorter than it should be 2011-03-28 15:23:40 -0700 1 1 1 Unclassified WebKit Forms 528+ (Nightly build) Mac OS X 10.5 RESOLVED DUPLICATE 34049 InRadar, Regression P1 Normal --- 1 john zwarich adele emacemac7 hyatt simon.fraser zwarich oldest_to_newest 108826 0 john 2009-02-07 08:06:13 -0800 Steps: 1. Try to drag thumb to the half way point. 2. Notice it is not actually at the half way point. 108827 1 27447 john 2009-02-07 08:06:48 -0800 Created attachment 27447 Demo 108879 2 ap 2009-02-08 02:11:53 -0800 Confirmed as a regression with r40771. The scroller is shorter than it should be, so it appears to be mispositioned when dragging. 108880 3 ap 2009-02-08 02:13:01 -0800 <rdar://problem/6566875> 109200 4 adele 2009-02-10 14:53:17 -0800 Adding Hyatt since he wrote http://trac.webkit.org/changeset/40771. 109356 5 adele 2009-02-11 13:21:05 -0800 Oh, Alexey clarified that 40771 is just the revision he used to confirm the bug, not the cause of the bug. 118459 6 zwarich 2009-04-22 18:53:02 -0700 This regressed between the r36847 and r36882 nightlies. It shouldn't be too hard to find the exact point of the regression, because it's such a small range. 118463 7 zwarich 2009-04-22 19:25:10 -0700 This regressed in r36869: http://trac.webkit.org/changeset/36869 118471 8 29704 zwarich 2009-04-22 21:18:36 -0700 Created attachment 29704 Proposed patch Do we usually add pixel tests for something like this? 118502 9 29704 darin 2009-04-23 09:07:09 -0700 Comment on attachment 29704 Proposed patch r=me And, yes, a manual test or a pixel test or both would be good to add. 119356 10 eric 2009-04-29 14:50:50 -0700 Assigning to cpst for test addition and landing. 126115 11 eric 2009-06-15 18:59:43 -0700 cpst? Ping? 127862 12 eric 2009-06-24 01:07:40 -0700 cpst: ping? Please advise. 128062 13 29704 eric 2009-06-24 18:35:33 -0700 Comment on attachment 29704 Proposed patch r- for lack of tests per darin's comment above. Can't land this. 374830 14 simon.fraser 2011-03-28 15:23:40 -0700 *** This bug has been marked as a duplicate of bug 34049 *** 27447 2009-02-07 08:06:48 -0800 2009-02-07 08:06:48 -0800 Demo select.html text/html 542 john PHNlbGVjdCBzaXplPSIyMCI+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+CjxvcHRpb24+YTwvb3B0aW9uPgo8b3B0aW9uPmE8L29wdGlvbj4K PG9wdGlvbj5hPC9vcHRpb24+Cgo8L3NlbGVjdD4= 29704 2009-04-22 21:18:36 -0700 2010-06-10 17:42:11 -0700 Proposed patch scrollbar.diff text/plain 1516 zwarich SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDQyNzY5 KQorKysgQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTggQEAKKzIwMDktMDQt MjIgIENhbWVyb24gWndhcmljaCAgPGN3endhcmljaEB1d2F0ZXJsb28uY2E+CisKKyAgICAgICAg UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQnVnIDIzODE5OiBSRUdSRVNT SU9OIChyMzY4NjkpOiBTZWxlY3Qgc2Nyb2xsZXIgaXMgc2hvcnRlciB0aGFuIGl0IHNob3VsZCBi ZQorICAgICAgICA8aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzODE5 PgorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNjU2Njg3NT4KKworICAgICAgICBUaGUgbmV3IE1h YyBzY3JvbGxiYXIgY29kZSB1c2luZyBISVRoZW1lRHJhd1RyYWNrIGFkZGVkIGluIHIzNjc3NyAo YnV0IAorICAgICAgICB0dXJuZWQgb24gaW4gcjM2ODY5KSBoYWQgYSB0eXBvLiBJdCBzZXQgdGhl IHZpZXcgc2l6ZSB0byB0aGUgdmFsdWUgb2YKKyAgICAgICAgc2Nyb2xsYmFyLT5wYWdlU3RlcCgp IGluc3RlYWQgb2Ygc2Nyb2xsYmFyLT52aXNpYmxlU2l6ZSgpLgorCisgICAgICAgICogcGxhdGZv cm0vbWFjL1Njcm9sbGJhclRoZW1lTWFjLm1tOgorICAgICAgICAoV2ViQ29yZTo6U2Nyb2xsYmFy VGhlbWVNYWM6OnBhaW50KToKKwogMjAwOS0wNC0yMiAgU2FtIFdlaW5pZyAgPHNhbUB3ZWJraXQu b3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IE1hcmsgUm93ZS4KSW5kZXg6IHBsYXRmb3JtL21h Yy9TY3JvbGxiYXJUaGVtZU1hYy5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBwbGF0Zm9ybS9tYWMvU2Nyb2xs YmFyVGhlbWVNYWMubW0JKHJldmlzaW9uIDQyNzYxKQorKysgcGxhdGZvcm0vbWFjL1Njcm9sbGJh clRoZW1lTWFjLm1tCSh3b3JraW5nIGNvcHkpCkBAIC0zNjcsNyArMzY3LDcgQEAgYm9vbCBTY3Jv bGxiYXJUaGVtZU1hYzo6cGFpbnQoU2Nyb2xsYmFyKgogICAgIHRyYWNrSW5mby5taW4gPSAwOwog ICAgIHRyYWNrSW5mby5tYXggPSBzY3JvbGxiYXItPm1heGltdW0oKTsKICAgICB0cmFja0luZm8u dmFsdWUgPSBzY3JvbGxiYXItPmN1cnJlbnRQb3MoKTsKLSAgICB0cmFja0luZm8udHJhY2tJbmZv LnNjcm9sbGJhci52aWV3c2l6ZSA9IHNjcm9sbGJhci0+cGFnZVN0ZXAoKTsKKyAgICB0cmFja0lu Zm8udHJhY2tJbmZvLnNjcm9sbGJhci52aWV3c2l6ZSA9IHNjcm9sbGJhci0+dmlzaWJsZVNpemUo KTsKICAgICB0cmFja0luZm8uYXR0cmlidXRlcyA9IDA7CiAgICAgaWYgKHNjcm9sbGJhci0+b3Jp ZW50YXRpb24oKSA9PSBIb3Jpem9udGFsU2Nyb2xsYmFyKQogICAgICAgICB0cmFja0luZm8uYXR0 cmlidXRlcyB8PSBrVGhlbWVUcmFja0hvcml6b250YWw7Cg==