[Webkit-unassigned] [Bug 64336] RenderListBox scroller doesn't scroll

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=64336


Joseph Pecoraro <joepeck at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |joepeck at webkit.org




--- Comment #1 from Joseph Pecoraro <joepeck at webkit.org>  2011-07-11 18:47:23 PST ---
Hmm, I see the "native animator" getting called, but ends up doing nothing:

    ScrollAnimatorMac::scroll...
    [m_scrollAnimationHelper.get() scrollToPoint:newPoint];

    gdb> po m_scrollAnimationHelper.get()
    <NSScrollAnimationHelper: 0x118616b50>

Because ScrollAnimatorMac::adjustScrollPositionIfNecessary resets the y scroll to 0:

    float newY = max<float>(min<float>(position.y(), m_scrollableArea->contentsSize().height() - m_scrollableArea->visibleHeight()), 0);
    => max(min(1, 125-129), 0)
    => max(min(1, -4), 0)
    => max(-1, 0)
    => 0

Is the "- m_scrollableArea->visibleHeight()" part of the equation necessary? It sounds like
we just want to clamp in the content size, in this case from 0 to 125 and the visible height
is not needed?

However, I think custom scrollers work. So I'll compare this to custom scrollers.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.