<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Thank you for your comments. Here is the updated webrev:</div><div class=""><br class=""></div><div class=""><a href="http://cr.openjdk.java.net/~serb/alans/8194327/webrev.01/" class="">http://cr.openjdk.java.net/~serb/alans/8194327/webrev.01/</a></div><div class=""><br class=""></div><div class="">Alan</div><div class=""><br class=""></div><br class=""><div><blockquote type="cite" class=""><div class="">On Apr 4, 2018, at 9:34 AM, Sergey Bylokhov <<a href="mailto:Sergey.Bylokhov@oracle.com" class="">Sergey.Bylokhov@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi, Alan.<br class="">A few comments about the test:<br class=""> - It is a mac specific and JtregNativeJdk should compile it on mac only<br class=""> - It should close all windows at the end, currently it leaves Finder opened.<br class=""> - it tries to use NSWindowStyleMask/NSWindowStyleMaskTitled which are available in  >10.12. We only plan to move to 10.9 soon. So the test should skip it or use NSInteger/NSTitledWindowMask for macOS < MAC_OS_X_VERSION_10_12.<br class=""> - It looks like other tests in JtregNativeJdk.gmk use libtest+Some useful name, I suggest to use the same instead of bugid(same for the test name "Test.java").<br class="">BUILD_JDK_JTREG_LIBRARIES_LIBS_libtest819432<br class=""><br class="">On 02/04/2018 19:35, Alan Snyder wrote:<br class=""><blockquote type="cite" class="">Please review the following change to the macOS AWT.<br class="">Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8194327" class="">https://bugs.openjdk.java.net/browse/JDK-8194327</a><br class="">Webrev: <a href="http://cr.openjdk.java.net/~serb/alans/8194327/webrev.00/" class="">http://cr.openjdk.java.net/~serb/alans/8194327/webrev.00/</a><br class="">The goal of this change is to allow a Java desktop application on macOS to properly coexist with a native utility panel, such as the native color chooser.<br class="">The native color chooser is an example of a window that can become the key (focused) window but cannot become the main window.<br class="">If the previously active window is a Java frame, it should resign key window status (lose focus), but retain the main window status.<br class="">A window that is main but not key does not own the keyboard focus, but it appears active, and if it is using the screen menu bar,<br class="">it may be invoked to process a menu item action (if the menu item is not already handled by the key window).<br class="">The current macOS AWT does not support this combination of window states. A Java window is either key and main, or neither.<br class="">When the color chooser becomes key (obtains focus), the Java frame resigns both key and main status.<br class="">This change allows the key window status to be resigned while retaining the main window status, with the appropriate behavior.<br class="">Note that with this change, it remains impossible to implement a Java window that behaves like the native color chooser (i.e., can become key but not main).<br class="">That would require a much bigger change.<br class="">   Alan<br class=""></blockquote><br class=""><br class="">-- <br class="">Best regards, Sergey.<br class=""></div></div></blockquote></div><br class=""></body></html>