RESOLVED DUPLICATE of bug 34049 23819
REGRESSION (r36869): Select scroller is shorter than it should be
https://bugs.webkit.org/show_bug.cgi?id=23819
Summary REGRESSION (r36869): Select scroller is shorter than it should be
John Moe
Reported 2009-02-07 08:06:13 PST
Steps: 1. Try to drag thumb to the half way point. 2. Notice it is not actually at the half way point.
Attachments
Demo (542 bytes, text/html)
2009-02-07 08:06 PST, John Moe
no flags
Proposed patch (1.48 KB, patch)
2009-04-22 21:18 PDT, Cameron Zwarich (cpst)
eric: review-
John Moe
Comment 1 2009-02-07 08:06:48 PST
Alexey Proskuryakov
Comment 2 2009-02-08 02:11:53 PST
Confirmed as a regression with r40771. The scroller is shorter than it should be, so it appears to be mispositioned when dragging.
Alexey Proskuryakov
Comment 3 2009-02-08 02:13:01 PST
Adele Peterson
Comment 4 2009-02-10 14:53:17 PST
Adding Hyatt since he wrote http://trac.webkit.org/changeset/40771.
Adele Peterson
Comment 5 2009-02-11 13:21:05 PST
Oh, Alexey clarified that 40771 is just the revision he used to confirm the bug, not the cause of the bug.
Cameron Zwarich (cpst)
Comment 6 2009-04-22 18:53:02 PDT
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.
Cameron Zwarich (cpst)
Comment 7 2009-04-22 19:25:10 PDT
Cameron Zwarich (cpst)
Comment 8 2009-04-22 21:18:36 PDT
Created attachment 29704 [details] Proposed patch Do we usually add pixel tests for something like this?
Darin Adler
Comment 9 2009-04-23 09:07:09 PDT
Comment on attachment 29704 [details] Proposed patch r=me And, yes, a manual test or a pixel test or both would be good to add.
Eric Seidel (no email)
Comment 10 2009-04-29 14:50:50 PDT
Assigning to cpst for test addition and landing.
Eric Seidel (no email)
Comment 11 2009-06-15 18:59:43 PDT
cpst? Ping?
Eric Seidel (no email)
Comment 12 2009-06-24 01:07:40 PDT
cpst: ping? Please advise.
Eric Seidel (no email)
Comment 13 2009-06-24 18:35:33 PDT
Comment on attachment 29704 [details] Proposed patch r- for lack of tests per darin's comment above. Can't land this.
Simon Fraser (smfr)
Comment 14 2011-03-28 15:23:40 PDT
*** This bug has been marked as a duplicate of bug 34049 ***
Note You need to log in before you can comment on or make changes to this bug.