RFR: 8295248: JEditorPane HTML form with multi selection broke drawing after reseting [v2]

Toshio Nakamura tnakamura at openjdk.org
Mon Oct 24 02:06:25 UTC 2022


On Fri, 21 Oct 2022 22:51:49 GMT, Damon Nguyen <dnguyen at openjdk.org> wrote:

>> Toshio Nakamura has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update testcase
>
> test/jdk/javax/swing/text/html/TestResetSelectForm.java line 93:
> 
>> 91:             if (getMaxColorDiff(baseColor, c) > LIMIT) {
>> 92:                 throw new RuntimeException("Unexpected List selection at " +
>> 93:                     (i+1) + ", " + baseColor + "," + c);
> 
> The points mentioned seem valid since I see the same things when I test it myself. OptionListModel's clearSelection is being called without the drawing updating. The fix seems logical and works as intended.
> 
> Just to nitpick, adding a space before and after the '+' in '(i+1)' here would match the pattern of all the other additions.

@DamonGuy Thank you for the comments. Fixed.

-------------

PR: https://git.openjdk.org/jdk/pull/10685



More information about the client-libs-dev mailing list