[OpenJDK 2D-Dev] [PATCH] 6636469: Java Fullscreen Exclusive Mode not working with Xorg server 1.3.0 and above

[Dan Munckton]

Hi all

Happy new year.

Although I've been silent for a bit, I'm still working on this. I've
tested more widely now and have managed to configure multi-monitor
setups on both Xorg server's 1.2 and 1.3. I'm also now on the SCA
contributors list.

UPDATED PATCH

Please see the attached updated patch.

I needed to add code to prevent FSEM (Fullscreen Exclusive Mode) on Xorg
1.3 with 2 or more monitors. Although I suspect this is technically
possible now (as it is in Windows) running a full-screen app in a
multi-monitor setup currently disables all Graphics devices except the
default and requires an X restart to bring them back. So I guess it'll
need more work to support that. 

Should I raise an RFE? Is this something worth doing?

FEEDBACK SO FAR

Guys, am I on the right lines?

Yuri, you were going to work on 6599351, does what I've done fit in or
were you planning more radical changes?

ADVICE ON JTREG APPROACH

Also I need some advice on how to approach a jtreg test for this. My
current plan is to use a shell script to query command line tools like
xdpyinfo and xrandr and then run a Java tool to check whether FSEM is or
isn't supported as expected. But I've no idea (yet) how portable this
would be across platforms and across versions of X.

I notice there aren't a great deal FSEM related tests in the source
other than one in jdk/test/java/awt/Fullscreen (which incidentally
appears to be missing a source file 'DisplayModeChanger.java').

I can see the benefit in providing a regression test for this but I'm
concerned that it's very dependent on the environment the tests are
being run on as to whether it'll get spotted at all.

Do you guys think it's a good idea to make a test for this or is it more
likely to be a pest to maintain?

Thanks

Dan Munckton
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 6636469_v2.diff
Type: text/x-patch
Size: 3709 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20080116/ebe7d3f6/6636469_v2.diff>