<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p>Gentle reminder for review.<br>
</p>
Regards<br>
Prasanta<br>
<div class="moz-cite-prefix">On 6/13/2016 4:31 PM, prasanta
sadhukhan wrote:<br>
</div>
<blockquote
cite="mid:d8d04706-3bf7-e803-6b3c-ba777b063785@oracle.com"
type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
<p><br>
</p>
<br>
<div class="moz-cite-prefix">On 6/13/2016 12:51 PM, prasanta
sadhukhan wrote:<br>
</div>
<blockquote
cite="mid:9676dbbd-3e5d-e80c-79db-c1fc740d2e6b@oracle.com"
type="cite">Hi All, <br>
<br>
Please review a fix for jdk9 where it was seen that if we try to
select some rows in a JTable, the text painted in the rows goes
missing. <br>
<br>
Bug: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8159068">https://bugs.openjdk.java.net/browse/JDK-8159068</a>
<br>
<br>
webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Epsadhukhan/8159068/webrev.00/">http://cr.openjdk.java.net/~psadhukhan/8159068/webrev.00/</a>
<br>
<br>
The issue was rMax value was decremented wrongly so when
paintCells() is called with wrong rMax, some rows were not
printed correctly. <br>
<br>
Fix is to make sure rmax is decremented properly, only when we
are trying to print whole visible portion of JTable and NOT when
some rows are being painted. <br>
<br>
Regarding the regression testcase, I could not make it automated
as the failure happens on random iteration. <br>
and also, getting selection background/foreground was giving
same values with and without the missing text. <br>
<br>
</blockquote>
Also, since it is a regression of <a moz-do-not-send="true"
href="https://bugs.openjdk.java.net/browse/JDK-8081491"
title="The case print incomplete." class="issue-link"
data-issue-key="JDK-8081491"><strike>8081491</strike></a>, it's
testcase are working fine with this fix and so did SwingSet2
JTable demo.<br>
<blockquote
cite="mid:9676dbbd-3e5d-e80c-79db-c1fc740d2e6b@oracle.com"
type="cite">Regards <br>
Prasanat <br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>