[webkit-reviews] review requested: [Bug 104764] Handling autoscroll in EventHandler should be re-factor : [Attachment 179212] Patch 5

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Dec 13 00:03:55 PST 2012


yosin at chromium.org has asked  for review:
Bug 104764: Handling autoscroll in EventHandler should be re-factor
https://bugs.webkit.org/show_bug.cgi?id=104764

Attachment 179212: Patch 5
https://bugs.webkit.org/attachment.cgi?id=179212&action=review

------- Additional Comments from yosin at chromium.org
Could you review this patch?
Thanks in advance.

= Changes since the last review =
* Move AutoscrollController class to its own files.
* Remove m_eventHandler from AutoscrollController
** Using m_autoscrollRenderer->frame()->eventHandler() if we needed.
* EventHandler::stopAutoscrollTimer() is just wrapper of
AutoscrollController::stopAutoscrollTimer().


More information about the webkit-reviews mailing list