Hello, Anthony.

Thank you for the review.

The updated version could be found here: 
http://cr.openjdk.java.net/~pchelko/9/7154841/webrev.01/

With best regards. Petr.

On 23.12.2013, at 23:10, Anthony Petrov <[email protected]> wrote:

> Hi Petr,
> 
> src/macosx/classes/sun/lwawt/macosx/CPlatformWindow.java
>> 621                         applyWindowLevel();
> 
> Here the applyWindowLevel() method should be invoked on the 'pw' instance 
> rather than on 'this'.
> 
> --
> best regards,
> Anthony
> 
> On 12/23/2013 07:21 PM, Petr Pchelko wrote:
>> Hello, AWT Team.
>> 
>> Please review the fix for the issue:
>> https://bugs.openjdk.java.net/browse/JDK-7154841
>> The fix is available at:
>> http://cr.openjdk.java.net/~pchelko/9/7154841/webrev/
>> 
>> The fix is simple: for popup windows we should use an NSPopupMenuWindowLevel 
>> level.
>> 
>> With best regards. Petr.
>> 

Reply via email to