<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-text-html" lang="x-unicode">
<div class="moz-text-flowed" style="font-family: -moz-fixed;
font-size: 14px;" lang="x-unicode">Hi Everyone, <br>
Please review the fix for 7181438.<br>
This bug was introduced in 7u4b11, when macosx-port was
integrated to jdku7 by this changeset:<br>
<a class="moz-txt-link-freetext" href="http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/9dfe50f456be">http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/9dfe50f456be</a><br>
<br>
Note that now we always set format to GL_RGBA. So opaque surface
has an alpha channel.<br>
We handle this situation in: OGLBlitSurfaceToSurface,
OGLBlitSwToSurface, OGLBlitToSurfaceViaTexture, <br>
but there is no check in OGLBlitSwToTexture().<br>
<br>
I assume that code for alpha verification should be the same.<br>
<br>
Bug: <a class="moz-txt-link-freetext"
href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7142091"></a><a class="moz-txt-link-freetext" href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7181438">http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7181438</a><br>
Webrev can be found at:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/7181438/webrev.00">http://cr.openjdk.java.net/~serb/7181438/webrev.00</a><br>
<br>
Also note that CR 7177173 and <font face="">7124244</font>
depends from this one.<br>
</div>
</div>
<pre class="moz-signature" cols="72">--
Best regards, Sergey. </pre>
</body>
</html>