Removing obsolete Aqua code

[Philip Race]

https://bugs.openjdk.org/browse/JDK-8344697 is closed as a dup.
At this point I'd say A, although re-opening and renaming is possible.

-phil


On 4/11/25 9:58 AM, Jeremy Wood wrote:
> in https://bugs.openjdk.org/browse/JDK-8344697 I pointed out that in 
> Mac's Aqua L&F: we’re using AncestorListeners to help repaint 
> JRootPane default buttons.
>
> Over a decade ago this logic was needed to emulate the native Aqua 
> experience, but since around 2016 the default button in Aqua does NOT 
> continually pulse.
>
> I’d like to remove this code, but I’m not sure how to proceed.
>
> Originally I planned to make a PR for 8344697 
> <https://bugs.openjdk.org/browse/JDK-8344697> that removes the unused 
> code. But after rebalancing the AWTEventMulticaster (in 
> https://bugs.openjdk.org/browse/JDK-8342782 ), the performance 
> problems observed in8344697 
> <https://bugs.openjdk.org/browse/JDK-8344697> do not reproduce.
>
> Should I:
> A. File a new ticket just to remove the obsolete code?
> B. Go ahead and name this new PR after8344697 
> <https://bugs.openjdk.org/browse/JDK-8344697> , even though that’s a 
> little misleading now?
> C. Don’t pursue this, because without a concrete problem / business 
> case the OpenJDK community shouldn’t focus on this tech debt.
>
> Thoughts?
>
> Regards,
>  - Jeremy
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/client-libs-dev/attachments/20250411/218de298/attachment-0001.htm>