Re: [7u6] Review request for 7182902 [macosx] Test api/java_awt/GraphicsDevice/indexTGF.html#SetDisplayMode fails on Mac OS X 10.7

[Andrew Brygin]

Hello Alexander,

 the fix looks fine to me.

Thanks,
Andrew

On 18.07.2012 16:31, Alexander Scherbatiy wrote:

Please review the fix:

bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7182902
webrev: http://cr.openjdk.java.net/~alexsch/7182902/webrev7u6.00/


This is a backported fix from the JDK 8 to JDK 7u6.

There are 3 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.

- IllegalArgumentException should be thrown in case if matching display mode is not found

Thanks,
Alexandr.