Looks fine.
On 16.03.16 20:03, Alexander Scherbatiy wrote:
Hello,
Could you review the updated fix:
http://cr.openjdk.java.net/~alexsch/8150844/webrev.02/
The GTKLookAndFeel setting is changed to the system L&F in the
HiDPIPropertiesUnixTest.
Thanks,
Alexandr.
On 16/03/16 20:34, Sergey Bylokhov wrote:
HiDPIPropertiesUnixTest will be noop on OSX:
try {
50 UIManager.setLookAndFeel(
51 "com.sun.java.swing.plaf.gtk.GTKLookAndFeel");
52 } catch (Exception e) {
53 return;
54 }
On 16.03.16 15:12, Alexander Scherbatiy wrote:
Could you review the updated fix:
http://cr.openjdk.java.net/~alexsch/8150844/webrev.01/
- the is2x() method is removed from the tests
- imports are updated
On 16/03/16 13:25, Alexander Stepanov wrote:
Hello Alexandr,
just a minor remark about is2x() in tests:
do we still need this method in MultiresolutionIconTest?
The following simple check can be used in doTest() instead:
boolean is2x = "2".equals(scale);
Color
expected = is2x ? C2X : C1X,
unexpected = is2x ? C1X : C2X;
the same, probably, for MenuMultiresolutionIconTest, but here it is a
matter of taste.
It seems also that the import for java.awt.geom.AffineTransform is
unused for Corrupted2XImageTest (and, in case of changes, in
MenuMultiresolutionIconTest).
Thanks,
Alexander
P.S. MultiresolutionIconTest: "@requires (os.family != "mac")" is
connected to JDK-8151303, so the test will fail on OS X until it is
fixed (having this line removed).
It should be fine because bug number 8151303 is mentioned in the @bug
tag and is counted as a known issue.
Thanks,
Alexandr.
On 3/16/2016 11:29 AM, Alexander Scherbatiy wrote:
Hello,
Could you review the fix:
bug: https://bugs.openjdk.java.net/browse/JDK-8150844
webrev: http://cr.openjdk.java.net/~alexsch/8150844/webrev.00
The sun.java2d.uiScale property is now used in CGraphicsDevice.
The requested tests are updated.
Thanks,
Alexandr.
--
Best regards, Sergey.