<AWT Dev> [9] Review Request: 8074763 Remove API references to java.awt.dnd.peer

[Sergey Bylokhov]

16.03.15 16:06, Kevin Rushforth wrote:
> The change looks fine to me. I tried to apply the patch (to verify 
> that it doesn't affect JFXPanel) but got merge conflicts with the 
> latest (as of today) jdk9/client/jdk repo. As long as Anton has 
> verified that it doesn't break compatibility with JFXPanel I am OK 
> with the change.
This fix depends on the Component.peer:
http://cr.openjdk.java.net/~serb/8074028/webrev.06/
>
> -- Kevin
>
>
> Sergey Bylokhov wrote:
>> Hello.
>> Please review the fix for jdk 9.
>> There are a number of public API which reference the unsupported 
>> java.awt. dnd.peer interfaces.
>>
>> protected 
>> java.awt.dnd.DragSource.createDragSourceContext(java.awt.dnd.peer.DragSourceContextPeer, 
>> ...)
>> public 
>> java.awt.dnd.DragSourceContext(java.awt.dnd.peer.DragSourceContextPeer, 
>> ...) constructor
>> public java.awt.dnd.DropTarget.addNotify(ComponentPeer peer) and 
>> removeNotify(ComponentPeer peer)
>> public java.awt.dnd.DropTargetContext.addNotify(DropTargetContextPeer 
>> dtcp)
>>
>> There is a decision to remove these references as described: 
>> http://mail.openjdk.java.net/pipermail/awt-dev/2015-February/008924.html
>> Changes description:
>>
>>   * DragSource.java, DragSourceContext.java, DropTarget.java : In all
>>     of these methods the peers are used as a parameters. In most of
>>     the cases these parameters are not necessary, because the peer
>>     can be accessed using the reference to the shared
>>     object(Component/DropTarget etc). Since these methods can be
>>     useful I did not remove them, but remove one parameter only.
>>   * DropTargetContext.java: addNotify() is called when we cannot get
>>     the information about a peer so I change type of the parameter
>>     and documentation of the method. It seems that these methods
>>     DropTargetContext.addNotify/removeNotify are not useful and I can
>>     change them by private version, but I don't know which way will
>>     be better.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8074763
>> Webrev can be found at: 
>> http://cr.openjdk.java.net/~serb/8074763/webrev.01
>>
>> -- 
>> Best regards, Sergey.
>>


-- 
Best regards, Sergey.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20150316/e5159e9a/attachment-0001.html>