<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="">Hi Sergey,<div class=""><br class=""></div><div class="">Added the test case from the previous iteration of the fix. Please review the webrev:</div><div class=""><a href="http://cr.openjdk.java.net/~mhalder/8136999/webrev.01/" class="">http://cr.openjdk.java.net/~mhalder/8136999/webrev.01/</a></div><div class=""><br class=""></div><div class="">Regards,</div><div class="">Manajit</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 26-Jul-2017, at 10:32 PM, 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 style="font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; font-family: 'Times New Roman'; font-size: 12pt;" class="">Hi, Manajit.<br class="">Can you please add a test from the previous iteration of the fix:<br class=""><a href="http://cr.openjdk.java.net/~alexsch/8136999/webrev.00" class="">http://cr.openjdk.java.net/~alexsch/8136999/webrev.00</a><br class=""><br class="">-----<span class="Apple-converted-space"> </span><a href="mailto:manajit.halder@oracle.com" class="">manajit.halder@oracle.com</a><span class="Apple-converted-space"> </span>wrote:<span class="Apple-converted-space"> </span><br class="">><span class="Apple-converted-space"> </span><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial">Hi All,</font><div class="">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial"><br class=""></font></div><div class="">><span class="Apple-converted-space"> </span><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69);">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial">Kindly review the fix for JDK10.</font></div><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69); min-height: 14px;">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial"><br class=""></font></div><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69);">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial">Bug: </font></div><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69);">><span class="Apple-converted-space"> </span><a href="https://bugs.openjdk.java.net/browse/JDK-8136999" class="" target="_blank"><font class="" size="2" face="Arial">https://bugs.openjdk.java.net/browse/JDK-8136999</font></a></div><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69); min-height: 14px;">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial"><br class=""></font></div><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69);">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial">Webrev:</font></div><div class="" style="margin: 0px; line-height: normal;">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial" color="#454545"><a href="http://cr.openjdk.java.net/~mhalder/8136999/webrev.00/" class="" target="_blank">http://cr.openjdk.java.net/~mhalder/8136999/webrev.00/</a></font></div><div class="" style="margin: 0px; line-height: normal;">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial"><br class=""></font></div><div class="" style="margin: 0px; line-height: normal;">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial">Fix:</font></div><div class="" style="margin: 0px; line-height: normal;">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial"><span class="" style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);">The flushEvent method is not required as mentioned in the previous review mail (</span><a href="http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010201.html)." class="" target="_blank" style="color: rgb(59, 115, 175); text-decoration: none; margin: 0px; padding: 0px; border: 0px; outline: 0px; vertical-align: baseline;">http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010201.html).</a><span class="" style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);"> </span></font></div><div class="" style="margin: 0px; line-height: normal;">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial"><span class="" style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);">flushEvent related functionalities were modified while fixing issue </span><a href="https://bugs.openjdk.java.net/browse/JDK-8006634" class="" target="_blank" style="color: rgb(59, 115, 175); text-decoration: none; margin: 0px; padding: 0px; border: 0px; outline: 0px; vertical-align: baseline;">JDK-8006634</a><span class="" style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);">  and hence deleted flushEvent method. </span></font></div></div><div class="">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial"><br class=""></font></div><div class="">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial">Regards,</font></div><div class="">><span class="Apple-converted-space"> </span><font class="" size="2" face="Arial">Manajit</font></div></div></div></div></blockquote></div><br class=""></div></body></html>