[Webkit-unassigned] [Bug 141597] REGRESSION: SVG does not support link dragging

[bugzilla-daemon at webkit.org]

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

Daniel Bates <dbates at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #250761|                            |review?
              Flags|                            |

--- Comment #4 from Daniel Bates <dbates at webkit.org> ---
Created attachment 250761
  --> https://bugs.webkit.org/attachment.cgi?id=250761&action=review
Patch and layout test

I chose to define elementIsLiveLink() as a non-member function because it is a convenience function and is implemented in terms of public interfaces. I was unclear if we should generalize this function for Node objects. I chose to have this function take a reference to Element object instead of a reference to a Node object because the concept of a "live link" (a hyperlink that can be clicked even if it is inside an editable region - maybe "is clickable hyperlink" would be better way to describe this concept?) is more related to higher-order UI behavior than the core behavior of a DOM Node. Let me know if it is preferred to put this functionality in Node or if there is a more appropriate place to put this functionality (HitTestResult?).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150415/baae916c/attachment-0001.html>