On Wed, 21 Dec 2022 13:38:31 GMT, Ambarish Rapte <ara...@openjdk.org> wrote:
>> Lukasz Kostyra has updated the pull request incrementally with one >> additional commit since the last revision: >> >> NGNode: Change comment at dirty bounds update for clarification > > modules/javafx.graphics/src/main/java/com/sun/javafx/sg/prism/NGNode.java > line 335: > >> 333: } else { >> 334: // TODO I think this is vestigial from Scenario and will >> never >> 335: // actually occur in real life... (RT-23956) > > The comments should be removed and updated to explain the scenario when flow > enters the else block and I think RT-23956 can be closed with this fix. I updated the comment. I agree, RT-23956 can now be closed. ------------- PR: https://git.openjdk.org/jfx/pull/978