[Webkit-unassigned] [Bug 67109] [EFL] Weak scroll feature

Mon Aug 29 05:23:07 PDT 2011

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

Raphael Kubo da Costa <kubo at profusion.mobi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kubo at profusion.mobi

--- Comment #2 from Raphael Kubo da Costa <kubo at profusion.mobi>  2011-08-29 05:23:07 PST ---
The API needs some love IMO.

 * ewk_frame_scroll_add now seems to have a complete different code path depending on whether weak scrolling is enabled, and one of the code paths now relies on ewk_view (so far ewk_frame calls ewk_view as little as possible, usually only to propagate signals). It's a bit hard to follow the code with those 1, 2 and 3-letter variables with similar names.
 * It was not clear at first sight what ewk_frame_weak_scroll_moving_viewport_get was supposed to do.
 * How is one expected to use this API in an application?
 * The purpose of this code seems to be to queue scroll requests and do them all at once at some point. If the performance gains are worthwhile, doesn't it make sense to make this the only possible behavior? How about processing the scroll requests in the main loop?
 * Does any other port implement this? If so, might it be something to add to WebCore?

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