[Webkit-unassigned] [Bug 91688] [chromium] Add CCScrollbarAnimationController class for compositor scrollbar animation

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jul 30 15:18:37 PDT 2012


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





--- Comment #17 from Adrienne Walker <enne at google.com>  2012-07-30 15:18:38 PST ---
(In reply to comment #13)

> I added early exit for CCLayerImpl::scrollBy so it becomes a no-op if newDelta == m_scrollDelta. We never set scrollMaximum on the layer in the test, thus scrollBy exits early and none of the layers are marked as changed.
> 
> Does it break any assumption to do an early exit in scrollBy?

That seems totally reasonable.

> I think we want to verify that the scrollbar will show in responding to the user gesture, and will eventually fade out as time goes. Is there a good way to simulate monotonicallyIncreasingTime()?

I'm not sure I understand what the problem is. You should just be able to pass arbitrary monotonically increasing time values to functions that need them.

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



More information about the webkit-unassigned mailing list