<AWT Dev> [9/8u40] Review request for RT-37149 and JDK-8049065 : Implement DnD for SwingNode

[Petr Pchelko]

Hello, Anthony.

The AWT part looks good to me. 

With best regards. Petr.

> On Aug 6, 2014, at 9:12 PM, Anthony Petrov <anthony.petrov at oracle.com> wrote:
> 
> Anton: thank you for reviewing the fix.
> 
> All: I need at least one more reviewer for the JDK part of the fix because we're going to back-port the change to 8u40. Could anyone please review it? Artem, Sergey, anyone?
> 
> For your convenience: https://javafx-jira.kenai.com/browse/RT-37149
> 
> http://cr.openjdk.java.net/~anthony/9-5.2/
> https://bugs.openjdk.java.net/browse/JDK-8049065
> 
> --
> best regards,
> Anthony
> 
> On 8/5/2014 5:08 PM, Anthony Petrov wrote:
>> Anton, Artem, Steve,
>> 
>> Could you please review this fix?
>> 
>> --
>> best regards,
>> Anthony
>> 
>> On 7/18/2014 6:44 PM, Anthony Petrov wrote:
>>> Hi Petr, Anton, Artem, Steve,
>>> 
>>> Please review the fix at https://javafx-jira.kenai.com/browse/RT-37149
>>> 
>>> Webrevs:
>>> 
>>> JDK: http://cr.openjdk.java.net/~anthony/9-5.2/
>>> 
>>> FX: http://cr.openjdk.java.net/~anthony/g-522-swingNodeDnD-RT-37149.3/
>>> 
>>> 
>>> JavaFX implements the DragSourceContextPeer and DragGestureRecognizer so
>>> that SwingNode content could pose as a drag source. In order to support
>>> a drop target, the DropTargetContextPeer is implemented in SwingNode and
>>> the add/removeDropTarget() methods register/unregister the drop target
>>> listeners.
>>> 
>>> The changes in JDK are mostly technical. We simply delegate the
>>> Toolkit.createDragSourceContextPeer() and
>>> Toolkit.createDragGestureRecognizer() factory methods to SwingNode.
>>> Similarly, we delegate the DropTargetPeer.add/removeDropTarget()
>>> operations to SwingNode as well.
>>> 
>>> In FX I've factored out the CachingTransferable class from the
>>> SwingDragSource class so as to share the implementation with the
>>> SwingNode DnD support. Also I've added a few utility methods to
>>> DataFlavorUtils and SwingFXUtils. The main fix is the new code in
>>> FXDnD.java which actually implements the AWT interfaces, installs
>>> appropriate event handlers on the SwingNode node in FX, and handles all
>>> the DnD events.
>>> 
>>> Note that the JDK <-> FX interface is loose because I use default
>>> methods in the LightweightContent interface, so that you can run new FX
>>> with the old JDK, or old FX with the new JDK, and nothing should break.
>>> Obviously, the DnD in SwingNode will only work if both JDK and FX are
>>> patched.
>>> 
>>> I've tested these changes on Windows and Mac with the sample code from
>>> this JIRA as well as a JFXPanel DnD test application from RT-34283. The
>>> DnD works fine for me in both intra- and inter-process modes.
>>> 
>>> Please post your review comments in JIRA.
>>> 
>>> --
>>> best regards,
>>> Anthony