Review request for 7142120: [macosx] Some JCK tests for SplashScreen fail on Mac OS X due to incorrect positioning of the splash (original) (raw)
Artem Ananiev artem.ananiev at oracle.com
Mon Feb 6 08:17:37 PST 2012
- Previous message: Review request for 7142120: [macosx] Some JCK tests for SplashScreen fail on Mac OS X due to incorrect positioning of the splash
- Next message: Review request for 7142120: [macosx] Some JCK tests for SplashScreen fail on Mac OS X due to incorrect positioning of the splash
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 2/3/2012 5:22 PM, Anthony Petrov wrote:
Hello,
Please review a fix for http://bugs.sun.com/viewbug.do?bugid=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
- Previous message: Review request for 7142120: [macosx] Some JCK tests for SplashScreen fail on Mac OS X due to incorrect positioning of the splash
- Next message: Review request for 7142120: [macosx] Some JCK tests for SplashScreen fail on Mac OS X due to incorrect positioning of the splash
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]