The fix looks good to me.
Thanks,
Alexandr.
On 8/29/2016 7:08 PM, Semyon Sadetsky wrote:
http://cr.openjdk.java.net/~ssadetsky/8160160/webrev.01/
return type of ENGINE.finishPainting() changed to Buffered image.
img.getRGB() returns a primitive not an object.
--Semyon
On 8/29/2016 3:45 PM, Alexandr Scherbatiy wrote:
On 8/29/2016 1:12 PM, Semyon Sadetsky wrote:
Hello,
Please review fix for JDK9:
https://bugs.openjdk.java.net/browse/JDK-8160160
bug: https://bugs.openjdk.java.net/browse/JDK-8160160
webrev: http://cr.openjdk.java.net/~ssadetsky/8160160/webrev.00/
The menu border is not painted by GTK Adwaita theme and makes popup
menu invisible at some circumstances. To resolve this the GTK menu
border color is compared to the menu background color and if the
same the border is painted by means of java graphics using a bit
different color tone.
- BufferedImage img = (BufferedImage) ENGINE.finishPainting();
if the ENGINE.finishPainting() always returns BufferedImage it
would be better to change the return type to BufferedImage.
- border == img.getRGB(w / 2, h / 2)
Could we rely on that img.getRGB(0, h / 2) returns the same object
as the img.getRGB(w / 2, h / 2)? May be it is use equals() methods
for comparing in this case.
Thanks,
Alexandr.
--Semyon
