
<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,<div class=""><br class=""></div><div class="">Thank you for your review comments. </div><div class=""><br class=""></div><div class="">I have modified the code as per your first two comments.</div><div class="">For the last comment the fix was tested with JDK versions b123, b124, b125, b127 and b130 and</div><div class="">the test failed with the popup message.</div><div class=""><br class=""></div><div class="">Please review the modified review.</div><div class=""><a href="http://cr.openjdk.java.net/~mhalder/8156099/webrev.01/" class="">http://cr.openjdk.java.net/~mhalder/8156099/webrev.01/</a></div><div class=""><br class=""></div><div class="">Thanks,</div><div class="">Manajit</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 09-Aug-2016, at 9:41 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="">Hi, Manajit.<br class=""> - Its seems that the test creates the Swing components on non-EDT.<br class=""> - The instructions are a little bit misleading, because popup will be shown even when the test is passed.<br class=""> - When I run the test on the current client ws, it is passed. Can you please double check that the test fails before the fix.<br class=""><br class="">On 01.08.16 12:35, Manajit Halder wrote:<br class=""><blockquote type="cite" class="">Hi All,<br class=""><br class="">Kindly review the fix for JDK9.<br class=""><br class="">Bug:<br class=""><a href="https://bugs.openjdk.java.net/browse/JDK-8156099" class="">https://bugs.openjdk.java.net/browse/JDK-8156099</a><br class=""><br class="">Webrev:<br class="">http://cr.openjdk.java.net/~mhalder/8156099/webrev.00/<br class=""><br class="">Issue:<br class="">[macosx] Drag and drop of link from web browser, DataFlavor types<br class="">application/x-java-url and text/uri-list, getTransferData returns null<br class=""><br class="">Cause:<br class="">This issue is a regression of issue<br class="">https://bugs.openjdk.java.net/browse/JDK-8136763.<br class="">While adding fix to 8136763 code was modified to handle coping of<br class="">multiple files but proper distinction was not added between bytes<br class="">containing files and urls.<br class=""><br class="">Fix:<br class="">Fixed by comparing format of the bytes received for dataflavor<br class="">"mimetype=text/uri-list".<br class="">Fixed by distinguishing between bytes containing files and urls. In both<br class="">cases the dataflavor contains same “mimetype=text/uri-list”, but the<br class="">format is different.<br class="">Format CF_FILE is only used in case of bytes containing files.<br class=""><br class="">Regards,<br class="">Manajit<br class=""></blockquote><br class=""><br class="">-- <br class="">Best regards, Sergey.<br class=""></div></blockquote></div><br class=""></div></body></html>