Bug 53628 - Clicking in scrollbar gutter of select multiple scrolls by only a single line, but should scroll by one "page"
Summary: Clicking in scrollbar gutter of select multiple scrolls by only a single line...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: data:text/html,<select%20multiple%20s...
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2011-02-02 14:09 PST by Adam Roben (:aroben)
Modified: 2011-05-31 22:46 PDT (History)
2 users (show)

See Also:


Attachments
patch (2.29 KB, patch)
2011-05-30 05:21 PDT, Keishi Hattori
no flags Details | Formatted Diff | Diff
test not working as expected (1.36 KB, text/html)
2011-05-31 18:53 PDT, Keishi Hattori
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2011-02-02 14:09:38 PST
To reproduce:

1. Go to data:text/html,<select%20multiple%20size=3><option>a<option>b<option>c<option>d<option>e<option>f<option>g<option>h<option>i<option>j<option>k<option>a<option>b<option>c<option>d<option>e<option>f<option>g<option>h<option>i<option>j<option>k<option>a<option>b<option>c<option>d<option>e<option>f<option>g<option>h<option>i<option>j<option>k<option>a<option>b<option>c<option>d<option>e<option>f<option>g<option>h<option>i<option>j<option>k
2. Click a few times in the lower part of the scrollbar gutter

Each click only scrolls by a single line. It should instead scroll by one "page", for some reasonable definition of "page".

This occurs in both Safari 5.0.3 and ToT.
Comment 1 Adam Roben (:aroben) 2011-02-02 14:10:39 PST
<rdar://problem/8950513>
Comment 2 Keishi Hattori 2011-05-30 05:21:27 PDT
Created attachment 95337 [details]
patch
Comment 4 Kent Tamura 2011-05-30 06:13:01 PDT
Comment on attachment 95337 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=95337&action=review

> Source/WebCore/ChangeLog:10
> +        Manual test: select-page-scroll.html
> +
> +        * manual-tests/select-page-scroll.html: Added.

I think platform-dependent test is better tan manual test.
We can write a test clicking a specific position, and put it to LayoutTests/platform/mac/fast/forms/
Comment 5 Keishi Hattori 2011-05-31 18:53:27 PDT
Created attachment 95531 [details]
test not working as expected

I've been trying to write a test using eventSender, but the scroll bar of select element is messed up in the test.

Pixel test draws the scrolls bar fine, but the up/down buttons don't work when clicked and paging doesn't work (s.scrollTop is 0). 

There is a 10x12 region in the top right corner of the select element that seems to act like a scroll bar. 

My test works fine on a regular div with scrollbars.

Is there something I can do?
Comment 6 Kent Tamura 2011-05-31 20:15:43 PDT
(In reply to comment #5)
> Created an attachment (id=95531) [details]
> test not working as expected

I also tried
 - -webkit-scrollbar instead of platform-native one
 - Testing in onload for <body>
 - Add some delay to start the test.
They didn't work.

ok, let's give up an automated test ;-(
Comment 7 WebKit Commit Bot 2011-05-31 22:46:53 PDT
Comment on attachment 95337 [details]
patch

Clearing flags on attachment: 95337

Committed r87773: <http://trac.webkit.org/changeset/87773>