<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Any feedback on this?<br>
    </p>
    Regards<br>
    Prasanta<br>
    <div class="moz-cite-prefix">On 8/16/2017 4:03 PM, Prasanta
      Sadhukhan wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:23734583-9a2e-be4a-a18c-fd1d08a8d23c@oracle.com">
      <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
      <p>Hi Sergey,<br>
      </p>
      AFAIK, FX singleThread feature is supported but experimental
      feature.<br>
      I have modified webrev to include DefaultKeyboardFocusManager too<br>
      <a class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Epsadhukhan/fx/8088132/webrev.01/"
        moz-do-not-send="true">http://cr.openjdk.java.net/~psadhukhan/fx/8088132/webrev.01/</a><br>
      <br>
      I do not think there is any problem in skipping code which works
      with EDT as for example, in SequencedEvent#dispatch() it says <br>
      <pre style="background-color:#ffffff;color:#000000;font-family:'Courier New';font-size:9.0pt;"><span style="color:#808080;font-style:italic;">Dispatches the nested event after all previous nested events have been</span><span style="color:#808080;font-style:italic;"> dispatched or disposed</span></pre>
      <div class="moz-cite-prefix">Now, since here FX App thread itself
        is the dispatch thread, we can be sure the events are dispatched
        in sequence and dispose is checked below after EDT.<br>
        <br>
        I have tested with couple of singleThread testcase without any
        issue.<br>
        <br>
        Regards<br>
        Prasanta<br>
        On 8/14/2017 10:07 PM, Sergey Bylokhov wrote:<br>
      </div>
      <blockquote type="cite"
        cite="mid:d95551ee-89e0-439c-9a7b-90ef9beb82d1@default">
        <pre wrap="">Hi, Prasanta, Kevin.

I have two notes.
 - Does this option is really supported? If it is supported we should evaluate all usage of EventDispatchThread because in this mode the dispatch thread is not EDT. For example I am not sure that we can skip the code which expects EventDispatchThread.
 - We have the similar pattern: "EventQueue.isDispatchThread() -> cast(EventDispatchThread)" in some other places like in DefaultKeyboardFocusManager.

----- <a class="moz-txt-link-abbreviated" href="mailto:prasanta.sadhukhan@oracle.com" moz-do-not-send="true">prasanta.sadhukhan@oracle.com</a> wrote:

</pre>
        <blockquote type="cite">
          <pre wrap="">Hi All,

Please review this fix
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Epsadhukhan/fx/8088132/webrev.00/" moz-do-not-send="true">http://cr.openjdk.java.net/~psadhukhan/fx/8088132/webrev.00/</a>
for an fx issue
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8088132" moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8088132</a>

More info in JBS.

Regards
Prasanta
</pre>
        </blockquote>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>