<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div dir="auto" style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div dir="auto" style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi All,<div class="">Please review the following fix for jdk16.</div><div class=""><br class=""></div><div class="">Bug : <a href="https://bugs.openjdk.java.net/browse/JDK-8041705" class="">https://bugs.openjdk.java.net/browse/JDK-8041705</a></div><div class="">Webrev : <a href="http://cr.openjdk.java.net/~trebari/swing/8041705/webrev00/" class="">http://cr.openjdk.java.net/~trebari/swing/8041705/webrev00/</a></div><div class=""><br class=""></div><div class="">Issue : The issue is that, border in DefaultTreeCellRenderer is not removed </div><div class="">if the margin is null.If we set the Nimbus LAF and then switch to metal LAF </div><div class="">nimbus border persists.</div><div class=""><br class=""></div><div class="">Fix : Added else part of the if(margins != null) , which will set border with margin</div><div style="orphans: 2; widows: 2;" class="">(0, 0, 0, 0) when the <span style="orphans: 2; widows: 2; background-color: rgb(255, 255, 255);" class=""><font color="#172b4d" face="DejaVu Sans, sans-serif" class="">DefaultLookup.getInsets(this, ui, "Tree.rendererMargins<span style="caret-color: rgb(23, 43, 77);" class="">â€</span>) returns null,</font></span></div><div style="orphans: 2; widows: 2;" class="">removing the previous margin.</div><div style="orphans: 2; widows: 2;" class=""><span style="orphans: 2; widows: 2; background-color: rgb(255, 255, 255);" class=""><font color="#172b4d" face="DejaVu Sans, sans-serif" class=""><span style="font-size: 14px;" class=""><br class=""></span></font></span></div><div style="orphans: 2; widows: 2;" class=""><span style="orphans: 2; widows: 2; background-color: rgb(255, 255, 255);" class=""><font color="#172b4d" face="DejaVu Sans, sans-serif" class="">Test : Added an automated test.</font></span></div><div style="orphans: 2; widows: 2;" class=""><span style="color: rgb(23, 43, 77); font-family: "DejaVu Sans", sans-serif; background-color: rgb(255, 255, 255);" class="">Tested on all the three platforms. Mach5 link is in JBS.</span></div><div style="orphans: 2; widows: 2;" class=""><span style="color: rgb(23, 43, 77); font-family: "DejaVu Sans", sans-serif; background-color: rgb(255, 255, 255);" class=""><br class=""></span></div><div style="orphans: 2; widows: 2;" class=""><span style="color: rgb(23, 43, 77); font-family: "DejaVu Sans", sans-serif; background-color: rgb(255, 255, 255);" class=""><br class=""></span></div><div style="orphans: 2; widows: 2;" class=""><font color="#172b4d" face="DejaVu Sans, sans-serif" class=""><span style="caret-color: rgb(23, 43, 77); background-color: rgb(255, 255, 255);" class="">Regards</span></font></div><div style="orphans: 2; widows: 2;" class=""><font color="#172b4d" face="DejaVu Sans, sans-serif" class=""><span style="caret-color: rgb(23, 43, 77); background-color: rgb(255, 255, 255);" class="">Tejpal</span></font></div></div></div></body></html>