Re: [9] Review Request for 8075907: The regression-swing case failed as the rollover icons is not correctly shown with the special options"-client -Dswing.defaultlaf=javax.swing.plaf.nimbus.NimbusLookAndFeel"

[Semyon Sadetsky]

On 9/17/2015 6:35 PM, Alexander Scherbatiy wrote:
On 9/17/2015 5:39 PM, Semyon Sadetsky wrote:
Hello,

Please review fix for JDK9:

bug: https://bugs.openjdk.java.net/browse/JDK-8075907
webrev: http://cr.openjdk.java.net/~ssadetsky/8075907/webrev.00/

The Nimbus L&F logic to choose the selected button rollover icon had 
a mistake.  The same for the selected button disabled icon.
    The state argument is never used in the 
SynthButtonUI.getIcon(sbstractButton, specificIcon, defaultIcon, 
state) method if defaultIcon is null.
    It looks like the suggested change never applies the 
SynthConstants.SELECTED state for non null defaultIcon.
That's true. Thanks. 
http://cr.openjdk.java.net/~ssadetsky/8075907/webrev.01/

    Thanks,
    Alexandr.

--Semyon