On 9/25/2013 16:37, Sergey Bylokhov wrote:
On 25.09.2013 16:32, Leonid Romanov wrote:
On 9/25/2013 16:09, Sergey Bylokhov wrote:
On 25.09.2013 15:49, Leonid Romanov wrote:
I'm not sure whether UIManager.getIcon("InternalFrame.closeIcon")
is guaranteed to return non null with other look&feels.
I am not sure it is guaranteed to wl&f as well. I don't see
specific to windows code.
com.sun.java.swing.plaf.windows.WindowsLookAndFeel has the following
line in initComponentDefaults() method:
"InternalFrame.closeIcon", WindowsIconFactory.createFrameCloseIcon()
So I think we can be sure about wl&f.
The same code exists in aqua, metal etc. But it can be null or can be
changed to null in future.
My assumption was that the set of UI defaults for a l&f is like a public
API for that l&f, in a sense that once you've added a particular default
you can't simply remove it later because of the backward compatibility.
On Sep 25, 2013, at 1:55 AM, Sergey Bylokhov
<sergey.bylok...@oracle.com> wrote:
Hi, Leonid.
Is the test not applicable for default l&f if WindowsLookAndFeel
is not exists?
On 25.09.2013 1:29, Leonid Romanov wrote:
Hello,
Please review a one line fix for 8016551: JMenuItem in
WindowsLookAndFeel can't paint default icons. I'm not sure
whether such a trivial change needs a regression test, but I've
written one anyway, just in case. If it's not needed, I won't
commit it.
bug: https://bugs.openjdk.java.net/browse/JDK-8016551
webrev: http://cr.openjdk.java.net/~leonidr/8016551/webrev.00/
<http://cr.openjdk.java.net/%7Eleonidr/8016551/webrev.00/>
Thanks,
Leonid.
--
Best regards, Sergey.