Review request for 7130935: [macosx] Still, JSpinner 4656590 regression (original) (raw)
Mike Swingler swingler at apple.com
Thu Jan 19 10:55:28 PST 2012
- Previous message: Review request for 7130935: [macosx] Still, JSpinner 4656590 regression
- Next message: [jdk7u-osx] Request for approval for 7122246: [macosx] JCK swing test CaretTests fails in b205
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On Jan 19, 2012, at 10:43 AM, Alexander Potochkin wrote:
Hello
Please review the following fix: http://cr.openjdk.java.net/~alexp/7130935/webrev.00/ the bug's description is here: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=7130935 The test case sets the font to JSpinner and then checks if the font from the DefaultEditor's textField is the same, attached In this fix I copied the missing block of code from the javax.swing.plaf.basic.BasicSpinnerUI class, lines 988-990 Thanks alexp <MyTest.java>
Looks good.
Thanks, Mike Swingler Apple Inc.
- Previous message: Review request for 7130935: [macosx] Still, JSpinner 4656590 regression
- Next message: [jdk7u-osx] Request for approval for 7122246: [macosx] JCK swing test CaretTests fails in b205
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]