On 2/3/2012 5:22 PM, Anthony Petrov wrote:
Hello,

Please review a fix for http://bugs.sun.com/view_bug.do?bug_id=7142120 at:

http://cr.openjdk.java.net/~anthony/x-14-splashScreen-7142120.0/

The fix is fairly simple: use NSScreen -frame instead of -visibleFrame.
In this case the splash screen is positioned in the geometrical center
of the screen. Visually this looks less pleasant (we may want to file an
RFE in the future), but makes tests happy which is what we need right now.

So what exactly do the tests expect? Does the test calculates the expected splashscreen position and compares to the value of SplashScreen.getBounds()? Does the latter method returns an incorrect value?

What I see in JavaDoc is just: "The window is positioned at the center of the screen." It's unclear if the center of the screen should or should not include screen insets.

Thanks,

Artem

All the failing tests now pass well with this fix applied.

--
best regards,
Anthony

Reply via email to