<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>On 24/05/16 15:22, Avik Niyogi wrote:<br>
</p>
<blockquote
cite="mid:ECD3DF0A-3FD3-4FE9-8C1E-5FF94C83D8CB@oracle.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
Hi All,
<div class=""><br class="">
</div>
<div class="">Kindly review the fix for JDK9.</div>
<div class=""><b class="">Bug</b>: <a moz-do-not-send="true"
href="https://bugs.openjdk.java.net/browse/JDK-7124218"
class="">https://bugs.openjdk.java.net/browse/JDK-8144161</a></div>
<div class=""><br class="">
</div>
<div class=""><b class="">Webrev</b>:<a class="moz-txt-link-freetext" href="http://">http://</a><a
moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eaniyogi/8144161/webrev.00/"
class="">cr.openjdk.java.net/~aniyogi/8144161/webrev.00/</a></div>
<div class=""><br class="">
</div>
<div class=""><b class="">Issue</b>: Test case throws an exception
when the behaviour is as expected.</div>
<div class=""><br class="">
</div>
<div class=""><b class="">Cause</b>: The expected behaviour for
comboBox was not accounted for Aqua look and feel in test case.</div>
<div class=""><br class="">
</div>
<div class=""><b class="">Fix</b>: The test case was fixed to
account for the Aqua LAF behaviour for comboBox dropdown menu
which extends beyond the screen Insets (the dock height) for
Mac.</div>
<div class=""><br class="">
</div>
</blockquote>
- the bug link points to the bug 7124218 instead of 8144161<br>
- The exception should be re-thrown on the line 63<br>
- Some code formatting change is not clear (see for example line
24 or 133)<br>
<br>
Thanks,<br>
Alexandr.<br>
<blockquote
cite="mid:ECD3DF0A-3FD3-4FE9-8C1E-5FF94C83D8CB@oracle.com"
type="cite">
<div class="">With Regards,</div>
Avik Niyogi
</blockquote>
<br>
</body>
</html>