<AWT Dev> [9] Review request for 8161195: Regression: closed/javax/swing/text/FlowView/LayoutTest.java

[Alexandr Scherbatiy]

The fix looks good to me.

Thanks,
Alexandr.

On 7/14/2016 1:05 AM, Semyon Sadetsky wrote:
> Hello,
>
> Please review fix for JDK9:
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8161195
>
> webrev: http://cr.openjdk.java.net/~ssadetsky/8161195/webrev.00/
>
> The IMN_SETCOMPOSITIONWINDOW message is generated by the OS upon 
> setting the composition window position and thus causes recursion by 
> requesting position.
>
> It should skip the position request as well as IMN_SETCANDIDATEPOS to 
> avoid recursion. But more correct solution is to request position of 
> the composition window only when it is made visible i.e. upon 
> IMN_OPENSTATUSWINDOW and WM_IME_STARTCOMPOSITION.
>
> Also javax/swing/text/FlowView/LayoutTest.java is moved to the open 
> regtest base.
>
> --Semyon
>