<AWT Dev> [9] Review request for 8179665: [Windows] java.awt.IllegalComponentStateException: component must be showing on the screen to determine its location

[Phil Race]

Approved.

-phil.

On 05/18/2017 10:37 AM, Semyon Sadetsky wrote:
>
> The correct link to webrev:
>
> http://cr.openjdk.java.net/~ssadetsky/8179665/webrev.01/
>
> Sorry...
>
> On 05/18/2017 10:31 AM, Semyon Sadetsky wrote:
>>
>> Hello,
>>
>> Please review fix for JDK9:
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8179665
>>
>> webrev: http://cr.openjdk.java.net/~ssadetsky/8179665/webrev.01/
>>
>> The root cause of the issue is IME events are sent when always-on-top 
>> state is set or cleared even during the window is not visible. In 
>> this case requesting the window position causes the exception.
>>
>> The fix introduces a check for window visibility before the 
>> calculation of the IME status window position. The latter should not 
>> be shown when the parent window is not visible so the IME window 
>> position is unnecessary.
>>
>> --Semyon
>>
>

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