<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi Sergey,<div class=""><br class=""></div><div class="">I think we can just replace ColorModel based calculation of the pixel value with SurfaceData.pixelFor(). The usage of ColorModel is intended for old Solaris platforms which are not supported any more. Please find the new version here: <a href="http://cr.openjdk.java.net/~dmarkov/8214109/webrev.01/" class="">http://cr.openjdk.java.net/~dmarkov/8214109/webrev.01/</a></div><div class=""><br class=""></div><div class="">Also I ran all regression tests and didn’t observe any new failures.</div><div class=""><br class=""></div><div class="">Thanks,</div><div class="">Dmitry<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 31 Jan 2019, at 22:25, Sergey Bylokhov <<a href="mailto:sergey.bylokhov@oracle.com" class="">sergey.bylokhov@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi, Dmitry.<br class="">On 30/01/2019 05:02, Dmitry Markov wrote:<br class=""><br class=""><blockquote type="cite" class="">I understand your intention to get rid of “the check of the current 2d pipeline” but it appears impossible to move the related code to java2d in particular OGL.<br class=""></blockquote><br class="">But it will be good to move it to java2d code, since this is ogl and solaris specific.(if this is really solaris specific then it looks like a bug in OGL pipeline)<br class=""><br class=""><blockquote type="cite" class="">Currently OGL uses ArgbPre pixel converter for rendering. Default pixel converter is used for calculation of pixel value when background colour is set because ArgbPre does not return the correct value for OGL on Solaris (according to JDK-6304250).<br class="">I do not see any way to distinguish between setting of background colour and other rendering operations from java2d code.<br class=""></blockquote><br class="">It is unclear why it is not possible to get this color since the current fix has a code to calculate this color.<br class=""><br class=""><br class="">-- <br class="">Best regards, Sergey.<br class=""></div></div></blockquote></div><br class=""></div></body></html>