+1 ----- philip.r...@oracle.com wrote: > 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 > >
