<Swing Dev> [9] RFR JDK-8159068:The rendering of JTable is broken

Alexandr Scherbatiy alexandr.scherbatiy at oracle.com
Tue Jun 21 11:46:01 UTC 2016


On 6/21/2016 1:58 PM, Prasanta Sadhukhan wrote:
>
>
> On 6/21/2016 4:14 PM, Alexandr Scherbatiy wrote:
>> On 6/20/2016 8:10 AM, prasanta sadhukhan wrote:
>>>
>>> Gentle reminder for review!!
>>>
>>> Regards
>>> Prasanta
>>> On 6/13/2016 4:31 PM, prasanta sadhukhan wrote:
>>>> On 6/13/2016 12:51 PM, prasanta sadhukhan wrote:
>>>>> Hi All,
>>>>>
>>>>> 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.
>>>>>
>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8159068
>>>>>
>>>>> webrev: http://cr.openjdk.java.net/~psadhukhan/8159068/webrev.00/
>>>>>
>>>>> The issue was rMax value was decremented wrongly so when 
>>>>> paintCells() is called with wrong rMax, some rows were not printed 
>>>>> correctly.
>>>>>
>>>>> 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.
>>
>>   Could you give two samples how this algorithm work. One sample 
>> where a whole visible portion of a JTable and another where some rows 
>> are being printed. What are rMax and  rMin values in both cases and 
>> how are they calculated?
>>
> If a JTable is of 50 rows and only 35 are being visible in page 1, then
> if whole visible portion of JTable is printed, rMin will be 0 and rMax 
> was 35
> so 36 rows were getting printed so I decrement rMax by 1 to 34 so only 
> 35 will be printed (same as shown on console).
> When we select some row of JTable as in the case of LostText testcase, 
> rMin will be say 6 and rMax will be 9 in which case also, I was 
> decrementing rMax so rMin=6, rMax=8 so next row was not getting painted.
    And what are indices of the selected rows?

   Thanks,
   Alexandr.

>
> Regards
> Prasanta
>>   Thanks,
>>   Alexandr.
>>
>>>>>
>>>>> Regarding the regression testcase, I could not make it automated 
>>>>> as the failure happens on random iteration.
>>>>> and also, getting selection background/foreground was giving same 
>>>>> values with and without the missing text.
>>>>>
>>>> Also, since it is a regression of 8081491 
>>>> <https://bugs.openjdk.java.net/browse/JDK-8081491>, it's testcase 
>>>> are working fine with this fix and so did SwingSet2 JTable demo.
>>>>> Regards
>>>>> Prasanat
>>>>
>>>
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20160621/dcd0bc26/attachment.html>


More information about the swing-dev mailing list