382205 - Clicking scrollbutton in <select> listbox scrolls by two <option>s

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect)

Description

[Jesse Ruderman]

Attachment: testcase

Steps to reproduce:
1. Load the testcase.
2. Click the down scrollbutton.

Result: scrolls by two options.

Expected: should scroll by one option.

Comment 1

[Josh Aas]

Can anyone reproduce this on Windows or Linux?

Comment 2

[[:Aleksej]]

WFM with Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a8pre) Gecko/2007091104 Minefield/3.0a8pre

Comment 3

[Carsten Book [:Tomcat]]

also wfm on Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9a8pre) Gecko/2007091005 Minefield/3.0a8pre ID:2007091005

Comment 4

[Josh Aas]

Attachment: fix v1.0

Dave Hyatt added code that messed this up in bug 129893 a long time ago. We should just kill it off.

Comment 5

[Colin Barrett [:cbarrett]]

Does this mean scrolling speed is decreased? We already scroll slower than Safari does.

Comment 6

[Josh Aas]

I don't think so. And it doesn't really matter, we can't be doing what is done in that ifdef anyway.

Comment 7

[Josh Aas]

Hyatt added a similar thing in presshell, see the "#ifdef MOZ_WIDGET_COCOA" there. I think getting rid of that might actually change scrolling speed in a way that matters, but this shouldn't.

Comment 8

[Colin Barrett [:cbarrett]]

Comment on attachment 284430 [details] [diff] [review]
fix v1.0

Ah, that's what I was thinking of when I read this patch, then.

Comment 9

[Josh Aas]

landed on trunk