<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=us-ascii"><meta name=Generator content="Microsoft Word 12 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";
color:black;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
p.MsoPlainText, li.MsoPlainText, div.MsoPlainText
{mso-style-priority:99;
mso-style-link:"Plain Text Char";
margin:0in;
margin-bottom:.0001pt;
font-size:10.5pt;
font-family:Consolas;}
span.PlainTextChar
{mso-style-name:"Plain Text Char";
mso-style-priority:99;
mso-style-link:"Plain Text";
font-family:Consolas;}
span.EmailStyle19
{mso-style-type:personal-compose;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]--></head><body lang=EN-US link=blue vlink=purple><div class=WordSection1><p class=MsoNormal><span lang=EN-IN>Hi<b>,</b></span><o:p></o:p></p><p class=MsoNormal style='mso-margin-top-alt:auto;mso-margin-bottom-alt:auto'><span lang=EN-IN>Please review fix for JDK9,</span><o:p></o:p></p><p class=MsoNormal><b><span lang=EN-IN style='color:#1F497D'>Bug:</span></b><span lang=EN-IN> </span><span lang=EN-IN style='color:#1F497D'><a href="https://bugs.openjdk.java.net/browse/JDK-6421058">https://bugs.openjdk.java.net/browse/JDK-6421058</a> </span><o:p></o:p></p><p class=MsoNormal><b><span lang=EN-IN style='color:#1F497D'>Webrev:</span></b><span lang=EN-IN style='color:#1F497D'> </span><span style='color:#1F497D'><a href="http://cr.openjdk.java.net/~pkbalakr/6421058/webrev.00/">http://cr.openjdk.java.net/~pkbalakr/6421058/webrev.00/</a> </span><span lang=EN-IN> </span><o:p></o:p></p><p class=MsoNormal><span lang=EN-IN> </span><o:p></o:p></p><p class=MsoNormal><b><span lang=EN-IN>Analysis:</span></b><o:p></o:p></p><p class=MsoNormal style='margin-left:.5in'><b>Expected Behavior: </b><br>1. The font of the text field is changed to the font of JSpinner if the font of text field was NOT set by the user. <br>2. The font of the text field is NOT changed to the font of JSpinner if the font of text field was set by the user. <o:p></o:p></p><p class=MsoNormal style='margin-left:.5in'><br>Default Font for JSpinner is set to "MetalTheme.CONTROL_TEXT_FONT", which is by default BOLD. <br>MetalLookAndFeel derives it's color palette and fonts from MetalTheme.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal style='margin-left:.5in'>As per Java docs: <br>DefaultMetalTheme uses bold fonts for many controls. To make all controls (with the exception of the internal frame title bars and client decorated frame title bars) use plain fonts you can do either of the following: <br><br> Set the system property swing.boldMetal to false. For example, java -Dswing.boldMetal=false MyApp. <br> Set the defaults property swing.boldMetal to Boolean.FALSE. For example: UIManager.put("swing.boldMetal", Boolean.FALSE);<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal style='margin-left:.5in'>As per analysis, Expected Behavior[2] holds good, where as Expected Behavior[1] fails to set the default font(i.e., Spinner's font) to TextField when TextField font is not set explicitly. <br><br><o:p></o:p></p><p class=MsoNormal><b>Issue:</b><br>Here the issue is other way, i.e., The Font in JSpinner is NOT BOLD, before calling JSpinner.setEditor().<o:p></o:p></p><p class=MsoNormal>Because TextField Font is NOT SET to JSpinner Font during Initialization.<o:p></o:p></p><p class=MsoNormal><span lang=EN-IN> </span><o:p></o:p></p><p class=MsoNormal><b><span lang=EN-IN>Fix:</span></b><o:p></o:p></p><p class=MsoNormal>TextField Font is SET to JSpinner Font during Initialization.<span lang=EN-IN> <o:p></o:p></span></p><p class=MsoNormal><span lang=EN-IN><o:p> </o:p></span></p><p class=MsoNormal><b><span lang=EN-IN>Regression Test:<o:p></o:p></span></b></p><p class=MsoPlainText>Jtreg and JCK tests PASSED without causing any regression with the suggested fix, across all platforms (Win/Linux/Mac).<o:p></o:p></p><p class=MsoNormal><b><span lang=EN-IN><o:p> </o:p></span></b></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal> <o:p></o:p></p><p class=MsoNormal><span lang=EN-IN>Regards,<br>Prem</span><o:p></o:p></p><p class=MsoNormal><span lang=EN-IN> </span><o:p></o:p></p><p class=MsoNormal><span style='color:windowtext'><o:p> </o:p></span></p></div></body></html>