Alexander,
couple of notes:
- where is the code that throws IllegalArgumentException in case of no
matching display mode found?
- similar way we have to add check that throws an exception when
someone tries to set display mode
when it's not allowed.
- the check for dcSupported here:
! if (originalMode != null) {
! if (dcSupported) {
! setDisplayMode(originalMode);
! }
! originalMode = null;
}
+ }
is not needed - later we check for it when we assign originalMode so if
originalMode is set to non-null value then
resolution change is allowed.
With best regards,
Alex
On 7/17/12 18:34, Alexander Scherbatiy wrote:
Hi,
Please review the fix:
bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7182902
webrev: http://cr.openjdk.java.net/~alexsch/7182902/webrev.00/
There are 2 fixed problems:
- GraphicsDevice can have display mode which has non zero refresh rate.
In this case the getBestModeForParameters method from the
CGraphicsDevice.m class should
return appropriate display mode even if the requested refresh rate
is zero.
- The original DisplayMode should be restored after exiting from the
Full Screen mode.
Thanks,
Alexandr.
