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://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 it to 8u66 it is too risky to change behavior. It should be reconsidered along with 8022810 [0].

[0] https://bugs.openjdk.java.net/browse/JDK-8022810 Cannot list all the available display modes on Ubuntu linux in case of two screen devices



--
Best regards, Sergey.

Reply via email to