On Mon, 23 Mar 2026 19:47:58 GMT, Marius Hanl <[email protected]> wrote:
>> @hjohn makes a good point, and I agree - timing tests are tricky. >> >> Another idea is maybe add a list change listener to the skin's >> `popup.getItems()` and check whether adding an N items triggers one change >> instead of N. What do you think? > >> Another idea is maybe add a list change listener to the skin's >> `popup.getItems()` and check whether adding an N items triggers one change >> instead of N. What do you think? > > That is indeed an easy but still effective idea for a test. I think we should > try it! Please see - whether the system test added in https://github.com/openjdk/jfx/pull/34 can be used as a reference while writing a test for this fix? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2118#discussion_r2980461303
