<html><head><style type='text/css'>p { margin: 0; }</style></head><body><div style='font-family: Times New Roman; font-size: 12pt; color: #000000'>Hi, Manajit.<br>Can you please add a test from the previous iteration of the fix:<br>http://cr.openjdk.java.net/~alexsch/8136999/webrev.00<br><br>----- manajit.halder@oracle.com wrote:
<br>> <div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">> <font class="" size="2" face="Arial">Hi All,</font><div class="">> <font class="" size="2" face="Arial"><br class=""></font></div><div class="">> <div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69);">> <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;">> <font class="" size="2" face="Arial"><br class=""></font></div><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69);">> <font class="" size="2" face="Arial">Bug: </font></div><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69);">> <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;">> <font class="" size="2" face="Arial"><br class=""></font></div><div class="" style="margin: 0px; line-height: normal; color: rgb(69, 69, 69);">> <font class="" size="2" face="Arial">Webrev:</font></div><div class="" style="margin: 0px; line-height: normal;">> <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;">> <font class="" size="2" face="Arial"><br class=""></font></div><div class="" style="margin: 0px; line-height: normal;">> <font class="" size="2" face="Arial">Fix:</font></div><div class="" style="margin: 0px; line-height: normal;">> <font class="" size="2" face="Arial"><span style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);" class="">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)." style="color: rgb(59, 115, 175); text-decoration: none; margin: 0px; padding: 0px; border: 0px; outline: 0px; vertical-align: baseline;" class="" target="_blank">http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010201.html).</a><span style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);" class=""> </span></font></div><div class="" style="margin: 0px; line-height: normal;">> <font class="" size="2" face="Arial"><span style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);" class="">flushEvent related functionalities were modified while fixing issue </span><a href="https://bugs.openjdk.java.net/browse/JDK-8006634" style="color: rgb(59, 115, 175); text-decoration: none; margin: 0px; padding: 0px; border: 0px; outline: 0px; vertical-align: baseline;" class="" target="_blank">JDK-8006634</a><span style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);" class="">  and hence deleted flushEvent method. </span></font></div></div><div class="">> <font class="" size="2" face="Arial"><br class=""></font></div><div class="">> <font class="" size="2" face="Arial">Regards,</font></div><div class="">> <font class="" size="2" face="Arial">Manajit</font></div></div></div></body></html>