Hello Sergey, The fix looks good to me too
-- Thanks, Alexander. On November 7, 2014 3:30:11 PM GMT+03:00, Alexander Scherbatiy <[email protected]> wrote: > > 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 >> -- Thanks, Alexander.
