RFR: 8277817: java/awt/dnd/BadSerializationTest/BadSerializationTest.java failed: ClassNotFoundException: com.apple.laf.AquaImageFactory$SystemColorProxy [v3]

[Kevin Rushforth]

On Thu, 2 Dec 2021 06:23:28 GMT, Liam Miller-Cushon <cushon at openjdk.org> wrote:

>>> I'm not sure I understand, aren't these being installed in `BasicPanelUI`?
>> 
>> If it is installs in the BasicListUI only then yes the fix belongs to this class. These properties changing also in some other classes as well, but some other classes cleanup it properly for example see BasicListUI.uninstallDefaults(), but I am not sure that all of them do this.
>
>> These properties changing also in some other classes as well, but some other classes cleanup it properly for example see BasicListUI.uninstallDefaults(), but I am not sure that all of them do this.
> 
> I looked through other uses of `installColorsAndFont`, and most of them don't seem to be uninstalling the properties. I updated the ones I found, and added a `uninstallColorsAndFont` method to avoid copying the logic around.

@cushon As a best practice, please do not force push to a branch for which there is an PR under review. Instead, you can merge the master branch into your branch if you need to update.

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

PR: https://git.openjdk.java.net/jdk/pull/6603