Re: [9] Review request for 8131752 [Regression] Test java/awt/GraphicsDevice/CheckDisplayModes.java fails

2015-07-23 Thread Alexander Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 7/22/2015 7:07 PM, Alexander Zvegintsev wrote: Hello, Please review the fix http://cr.openjdk.java.net/~azvegint/jdk/9/8131752/00/ for the issue https://bugs.openjdk.java.net/browse/JDK-8131752 This fix simply restores fullscreen windows chec

Re: [9] Review request for 8131752 [Regression] Test java/awt/GraphicsDevice/CheckDisplayModes.java fails

2015-07-22 Thread Sergey Bylokhov
The fix looks fine. It is unclear why in previous fix the check for fullscreen window was removed, but the check for fullscreen support not. On 22.07.15 19:07, Alexander Zvegintsev wrote: Hello, Please review the fix http://cr.openjdk.java.net/~azvegint/jdk/9/8131752/00/ for the issue https:/

[9] Review request for 8131752 [Regression] Test java/awt/GraphicsDevice/CheckDisplayModes.java fails

2015-07-22 Thread Alexander Zvegintsev
Hello, Please review the fix http://cr.openjdk.java.net/~azvegint/jdk/9/8131752/00/ for the issue https://bugs.openjdk.java.net/browse/JDK-8131752 This fix simply restores fullscreen windows check. Actually fullscreen is not necessary to change display mode, but since I am planning to backport