Push request: 7029903 Splash screen is not shown in 64-bit Linux with 16-bit color depth

[David Buck]

I'm sure the webrev will make the change easier to review...

http://cr.openjdk.java.net/~dbuck/7029903/webrev.01/

Sorry, not sure how I managed to forget the most important part of my 
request.

Regards,
-Buck

On 08/03/11 23:35, David Buck wrote:
> Hi!
>
> This is a request to backport a jdk8 fix into jdk7u2.
>
> CR:7029903 Splash screen is not shown in 64-bit Linux with 16-bit color
> depth
>
> Weblink: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=2208636
> (for some reason, the URL for the parent bug is not working. Sub CR (I
> believe that 2208636 will have all needed information.)
>
> Description:
> On X11, if image data for splash screen is small enough, it can
> remain in the Xlib client buffer until the JVM starts up and starts
> updating the GUI. I added an explicit call to Xflush() immediately
> after the splash screen image is updated.
>
> This is a regression introduced by the fix for cr6557810.
>
>
> The fix is already included in jdk8 as:
>
> Changeset: f7dd470d3794
> Author: dbuck
> Date: Wed Aug 03 07:04:34 2011 -0700
> URL: http://hg.openjdk.java.net/jdk8/swing/jdk/rev/f7dd470d3794
>
> 7029903: Splash screen is not shown in 64-bit Linux with 16-bit color depth
> Summary: Added Xflush() call after splash screen is updated to ensure
> update is no stuck in client side buffer until JVM starts up. See JET
> review request 4154 for details.
> Reviewed-by: kevinw, anthony
>
> Original jdk code review was done with JET (inernal Oracle code review
> tool. Will use correct review procedure from now on.).
>
> The patch for jdk7u2 is identical to the one in jdk8.
>
> Cheers,
> -Buck