<AWT Dev> [OpenJDK 2D-Dev] [9] Review Request: 8074028 Remove API references to java.awt.peer

[Anton Tarasov]

Thanks for the check, Kevin!

Anton.

On 06/03/15 02:13, Kevin Rushforth wrote:
> I verified that this doesn't break JavaFX Swing interop (specifically 
> Drag and Drop), so it looks fine to me.
>
> -- Kevin
>
>
> Sergey Bylokhov wrote:
>> The new version of the fix:
>> http://cr.openjdk.java.net/~serb/8074028/webrev.06
>> Font.getPeer() was renamed to Font.getFontPeer().
>>
>>
>> On 04.03.2015 15:51, Alan Bateman wrote:
>>> On 04/03/2015 12:37, Sergey Bylokhov wrote:
>>>> Hello.
>>>> Please review the fix for jdk 9.
>>>> There are a number of public API whichreference the unsupported 
>>>> java.awt.peer interfaces.
>>>>
>>>> public java.awt.Component.getPeer() @deprecated 1.1
>>>> public java.awt.Font.getPeer() @deprecated 1.1
>>>> public java.awt.MenuComponent.getPeer() @deprecated 1.1
>>>>
>>> I don't know this area so I don't feel qualified to do a thorough 
>>> review of the changes. However I welcome the changes, this is long 
>>> standing technical debt and important that we address this as part 
>>> of the move to modules.
>>>
>>> I note that Font::getPeer is changed to private, hence something try 
>>> to access it outside of the Font class will get an 
>>> IllegalAccessError as opposed to NoSuchMethodError for the outright 
>>> removal in the other cases. Did you consider renaming to something 
>>> else, say private FontPeer getFontPeer so that it is consistent with 
>>> the removal of the other two methods?
>>>
>>> I note that you might not have caught all tests but I would assume 
>>> that any missed tests should fail. Perhaps there are other failures 
>>> that are potentially hiding tests using getPeer?
>>>
>>> -Alan
>>>
>>
>>