The fix looks good to me.
Thanks,
Alexandr.
On 11/7/2014 4:33 PM, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk 9.
The problem is that BasicLookAndFeel (SynthILookAndFeel as well) has
its own component hierarchy on top of the InternalFrameUI. When text
for these components are painted in SwingUtilities2.drawString() the
client property AA_TEXT_PROPERTY_KEY is checked.
For normal components this property is initialized in the setUI(), but
this method is not called for a ui component itself. In the fix I
transfer this property from component to the ui delegate.
The test is absent, because all of this stuff is not public-> not
accessible in modular world(AA_TEXT_PROPERTY_KEY, SwingUtilities2.*)
Bug: https://bugs.openjdk.java.net/browse/JDK-7169583
Webrev can be found at:
http://cr.openjdk.java.net/~serb/7169583/webrev.00