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


Reply via email to