<div dir="ltr"><div dir="ltr"><div dir="ltr">Even though I'm still not convinced about the need of imposing partial order sync on asynchronous events, we can do it.<div><br></div><div>Here it's Webrev.03 that includes partial ordering of non-SequencedEvent events and Sergey's unit test:</div><div><br></div><div> * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/8204142/8204142.webrev.03/">http://cr.openjdk.java.net/~mbalao/webrevs/8204142/8204142.webrev.03/</a></div><div> * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/8204142/8204142.webrev.03.zip">http://cr.openjdk.java.net/~mbalao/webrevs/8204142/8204142.webrev.03.zip</a></div><div><br></div><div>I've put Sergey's unit test into a bash-loop and reached 300+ iterations without any issues. Laurent's functional test is also passing. Laurent's unit test (not included here) does not pass for the reasons already discussed.</div><div><br></div><div>Kind regards,</div><div>Martin.-</div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 31, 2018 at 1:13 PM, Sergey Bylokhov <span dir="ltr"><<a href="mailto:Sergey.Bylokhov@oracle.com" target="_blank">Sergey.Bylokhov@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi, Martin.<span class=""><br>
On 31/10/2018 09:03, Martin Balao wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Your MultipleContextsUnitTest test has 2 assertions that don't look good to me:<br>
<br>
  * dispatchSENumber < num1<br>
  * dispatchSENumber < num2<br>
<br>
My understanding is that these assertions mean that a non-SequencedEvent event is expected to be synchronized with SequencedEvent events. If such synchronization is needed, the event has to be wrapped in a SequencedEvent event. There are no guarantees otherwise; previous to my proposal these event were discarded and in my Webrev.02 they are dispatched asynchronously.<br>
</blockquote>
<br></span>
This is not a strong synchronization, it is just an expectation that the events which were posted after SequencedEvent should be dispatched after it. So if the app will have focus event and then mouse click, then mouse click should be dispatched after the focus. Note that the for this case the test does not check the exact sequence of order(==), just a relative order(<).<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
-- <br>
Best regards, Sergey.<br>
</font></span></blockquote></div><br></div>