Hi Sergey,
I suggest to add a comment/javadoc to
CWrapper.NSWindow.setBackgroundColor() to document the format of the
integer argument (the order of the color components, to be precise, or
the color model.) No need for a new webrev with this change.
Other than that, the fix looks good to me.
--
best regards,
Anthony
On 6/11/2014 5:13 PM, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk 9, which is also targeted for jdk 8u20.
Problems description:
JDialog does not set background color of the peer, like JFrame does +
there is a lag between the showing of the window and the first repaint
action(when the swing paints its own background). We should change the
color of the native window to the color of the Component to fix flashing.
I need a review a swing and awt part.
Bug: https://bugs.openjdk.java.net/browse/JDK-8033786
Webrev can be found at: http://cr.openjdk.java.net/~serb/8033786/webrev.00
