Bug 23819

Summary: REGRESSION (r36869): Select scroller is shorter than it should be
Product: WebKit Reporter: John Moe <john>
Component: FormsAssignee: Cameron Zwarich (cpst) <zwarich>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: adele, emacemac7, hyatt, simon.fraser, zwarich
Priority: P1 Keywords: InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Demo
none
Proposed patch eric: review-

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
Details
Proposed patch (1.48 KB, patch)
2009-04-22 21:18 PDT, Cameron Zwarich (cpst) 		eric: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
John Moe
Comment 1 2009-02-07 08:06:48 PST
Created attachment 27447 [details] Demo
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
<rdar://problem/6566875>
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
This regressed in r36869: http://trac.webkit.org/changeset/36869
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.