WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Proposed patch
(1.48 KB, patch)
2009-04-22 21:18 PDT
,
Cameron Zwarich (cpst)
eric
: review-
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug