<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi Sergey, Phil and Kevin,<div class=""><br class=""></div><div class="">Thank you all for your review comments.</div><div class=""><br class=""></div><div class="">Changed the fix as per Sergey’s review comment. Now both Frame and Dialog are checked for resizibility before setting canFullScreen to true.</div><div class="">Please review the modified webrev:</div><div class=""><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span style="font-variant-ligatures: no-common-ligatures" class=""><a href="http://cr.openjdk.java.net/~mhalder/8190767/webrev.03/" class="">http://cr.openjdk.java.net/~mhalder/8190767/webrev.03/</a></span></div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span style="font-variant-ligatures: no-common-ligatures" class=""><br class=""></span></div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">Thanks,</span></div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">Manajit</span></div><div class=""><br class=""></div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 17-Feb-2018, at 8:03 AM, Manajit Halder <<a href="mailto:manajit.halder@oracle.com" class="">manajit.halder@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=utf-8" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Correcting the subject to indicate that the fix is for JDK 11.<div class=""><div class=""><br class=""></div><div class="">Regard,</div><div class="">Manajit</div><div class=""><br class=""><div class=""><blockquote type="cite" class=""><div class="">On 16-Feb-2018, at 9:48 PM, Kevin Rushforth <<a href="mailto:kevin.rushforth@oracle.com" class="">kevin.rushforth@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">

  <meta content="text/html;charset=UTF-8" http-equiv="Content-Type" class="">
  <title class=""></title>

<div bgcolor="#ffffff" text="#000000" class="">
Perhaps the subject should be changed to indicate that the fix is for
JDK 11 (not JDK 10 for which it is way too late)...<br class="">
<br class="">
-- Kevin<br class="">
<br class="">
<br class="">
Manajit Halder wrote:
<blockquote cite="mid:CD0F2B6C-5300-41DD-AED3-15D413D01429@oracle.com" type="cite" class="">
  <meta http-equiv="Content-Type" content="text/html " class="">
Hi Sergey,
  <div class=""><br class="">
  <div class="">Please review the modified fix. The current webrev
maintains the window behaviour after it is shown as per the current
implementation. </div>
  <div class="">canFullScreen is set true in case the window is
  <div class=""><br class="">
  <div class=""><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Emhalder/8190767/webrev.02/" class="">http://cr.openjdk.java.net/~mhalder/8190767/webrev.02/</a></div>
  <div class=""><br class="">
  <div class="">Regards,</div>
  <div class="">Manajit</div>
  <div class=""><br class="">
  <div class="">
  <blockquote type="cite" class="">
    <div class="">On 14-Feb-2018, at 8:29 AM, Sergey Bylokhov <<a moz-do-not-send="true" href="mailto:sergey.bylokhov@oracle.com" class="">sergey.bylokhov@oracle.com</a>> wrote:</div>
    <br class="Apple-interchange-newline">
    <div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; float: none; display: inline ! important;" class="">Hi, Manajit.><span class="Apple-converted-space"> </span></span><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Emhalder/8190767/webrev.01/" class="">http://cr.openjdk.java.net/~mhalder/8190767/webrev.01/</a><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">
    <span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; float: none; display: inline ! important;" class="">After the window is shown behavior should be the same as in
current implementation(resizable windows should have
canFullScreen=true, others canFullScreen=false) so you cannot just set
it to "true" or "false".</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">
    <br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">
    <blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">The issue is not reproducible in jdk8. ><br class="">
Regards,<br class="">
Manajit<br class="">
      <blockquote type="cite" class=""><br class="">
On 31-Jan-2018, at 2:40 AM, Sergey Bylokhov <<a moz-do-not-send="true" href="mailto:sergey.bylokhov@oracle.com" class="">sergey.bylokhov@oracle.com</a> <<a moz-do-not-send="true" href="mailto:sergey.bylokhov@oracle.com" class="">mailto:sergey.bylokhov@oracle.com</a>>>
wrote:<br class="">
        <br class="">
Hi, Manajit.<br class="">
Did you check that it works properly in jdk8?<br class="">
        <br class="">
I guess we need to set "WINDOW_FULLSCREENABLE" after we show the window
for the first time, with one exception if the window is MAXIMIZED_BOTH.<br class="">
        <br class="">
On 29/01/2018 04:39, Manajit Halder wrote:<br class="">
        <blockquote type="cite" class="">Hi All,<br class="">
Kindly review the fix for JDK10.<br class="">
Bug:<br class="">
          <a moz-do-not-send="true" href="https://bugs.openjdk.java.net/browse/JDK-8190767" class="">https://bugs.openjdk.java.net/browse/JDK-8190767</a><br class="">
Webrev:<br class="">
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~mhalder/8190767/webrev.00/">http://cr.openjdk.java.net/~mhalder/8190767/webrev.00/</a><br class="">
Problem:<br class="">
In this problem two unrelated windows are created (first window and
second window). If the second window is created when the first window
is in fullscreen then the second window will automatically be created
in fullscreen mode. This is the default behaviour with Cocoa windows.
The second window receives windowWillEnterFullScreen notification when
the first window is in fullscreen. windowWillEnterFullScreen
notification is system generated notification and there is no way to
prevent it.<br class="">
Above conclusion was made after debugging the issue to find out if
there is any way to prevent the windowWillEnterFullScreen notification
and also to find out if it generated due to some existing java (generic
and native) code.<br class="">
I wrote a simple Mac OS X application with 2 windows and observed the
same behaviour. The second window is created in fullscreen mode if it
is created when the first window is in fullscreen. Whereas the second
window is displayed normal if the first window is in normal mode (not
in fullscreen mode). The only way found to prevent the second window
going to fullscreen is don’t set this behaviour for the second window.<br class="">
Fix:<br class="">
The behaviour is normal on Mac OS. But if we want to prevent the second
(all windows except the primary window) window to automatically created
in fullscreen mode then the following fix can be applicable.<br class="">
By default all the Frames are set the WINDOW_FULLSCREENABLE property
and by default all frames receives fullscreen event if the first frame
is in fullscreen when it is created. Due to this setting the second
frame which is created on button click goes to fullscreen on creation
automatically. Mac OS fullscreen event notifications are received in
case the first frame is in fullscreen and WINDOW_FULLSCREENABLE set for
the second frame.<br class="">
Only the first frame should have the WINDOW_FULLSCREENABLE property
set. The frames created in this case were ownerless windows as
mentioned earlier. Therefore it is not possible to find out the owner
of the current frame, it is null for all the frames created. Hence the
fix is if the current frame is the first frame then set the property
otherwise don’t set the property WINDOW_FULLSCREENABLE.<br class="">
Regards,<br class="">
Manajit<br class="">
        <br class="">
        <br class="">
--<span class="Apple-converted-space"> </span><br class="">
Best regards, Sergey.<br class="">
    <br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">
    <br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">
    <span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; float: none; display: inline ! important;" class="">--<span class="Apple-converted-space"> </span></span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;" class="">
    <span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; float: none; display: inline ! important;" class="">Best regards, Sergey.</span></div>
  <br class="">

</div></blockquote></div><br class=""></div></div></div></div></blockquote></div><br class=""></div></div></body></html>