[webkit-reviews] review requested: [Bug 39725] Dragging over an element with scrollbars should scroll the element when dragging near edges : [Attachment 182711] Patch-3.0.2

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jan 15 00:37:51 PST 2013


yosin at chromium.org has asked  for review:
Bug 39725: Dragging over an element with scrollbars should scroll the element
when dragging near edges
https://bugs.webkit.org/show_bug.cgi?id=39725

Attachment 182711: Patch-3.0.2
https://bugs.webkit.org/attachment.cgi?id=182711&action=review

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

This patch fixes non-stopping autoscroll issue. The root cause is forgetting
call of stop autscroll timer at cancellation of drag-and-drop, e.g. pressing
ESCAPE key, drag off the window from another process's drag source.

= Changes since the last review =
* Call stopAutoscrollTimer() at EventHandler::clearDragState() which called
after drop or cancel of drag-and-drop.
* Introduce ChromeClient::shouldAutoscrollForDragAndDrop() for enabling
drag-and-drop autoscroll by each port rater than all ports. Note: Chromium port
enables.
* Layout test drag-and-drop-autosccroll.html checks stopping of autoscroll
after drag-and-drop cancellation.


More information about the webkit-reviews mailing list