<html><head>
<style id="css_styles">
blockquote.cite { margin-left: 5px; margin-right: 0px; padding-left: 10px; padding-right:0px; border-left: 1px solid #cccccc }
blockquote.cite2 {margin-left: 5px; margin-right: 0px; padding-left: 10px; padding-right:0px; border-left: 1px solid #cccccc; margin-top: 3px; padding-top: 0px; }
a img { border: 0px; }
li[style='text-align: center;'], li[style='text-align: center; '], li[style='text-align: right;'], li[style='text-align: right; '] { list-style-position: inside;}
body { font-family: Helvetica; font-size: 9pt; }
.quote { margin-left: 1em; margin-right: 1em; border-left: 5px #ebebeb solid; padding-left: 0.3em; }
a.em-mention[href] { text-decoration: none; color: inherit; border-radius: 3px; padding-left: 2px; padding-right: 2px; background-color: #e2e2e2; }
._em_placeholder {color: gray; border-bottom: 1px dotted lightblue;} ._em_placeholder:before{color:gray; content: '{{ ';} ._em_placeholder:after{color:gray; content: ' }}';}
</style>
</head>
<body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">in <a href="https://bugs.openjdk.org/browse/JDK-8344697">https://bugs.openjdk.org/browse/JDK-8344697</a> I pointed out that in Mac's Aqua L&F: we’re using AncestorListeners to help repaint JRootPane default buttons.<div><br /></div><div>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.</div><div><br /></div><div>I’d like to remove this code, but I’m not sure how to proceed.</div><div><br /></div><div>Originally I planned to make a PR for <a href="https://bugs.openjdk.org/browse/JDK-8344697" style="font-size: 9pt;">8344697</a> that removes the unused code. But after rebalancing the AWTEventMulticaster (in <a href="https://bugs.openjdk.org/browse/JDK-8342782" style="font-size: 9pt;">https://bugs.openjdk.org/browse/JDK-8342782</a> ), the performance problems observed in<span> </span><a href="https://bugs.openjdk.org/browse/JDK-8344697" style="font-size: 9pt; margin: 0px;">8344697</a> do not reproduce.</div><div><br /></div><div>Should I:</div><div>A. File a new ticket just to remove the obsolete code?</div><div>B. Go ahead and name this new PR after<span> </span><a href="https://bugs.openjdk.org/browse/JDK-8344697" style="font-size: 9pt; margin: 0px;">8344697</a> , even though that’s a little misleading now?</div><div>C. Don’t pursue this, because without a concrete problem / business case the OpenJDK community shouldn’t focus on this tech debt.</div><div><br /></div><div>Thoughts?</div><div><br /></div><div>Regards,</div><div> - Jeremy</div></body></html>