On Tue, 9 Jul 2024 11:38:30 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> Another possibility is the code that measures the time it takes to show the >> tooltip. >> >> The current code uses Util.waitForLatch(), L244 which was written for a >> different use case and actually introduces a small measurement error as it >> includes the time needed for the context switch. Perhaps a better solution >> would be to note the timestamp in L271 where the tooltip listener gets >> called (similar to the way the start moment is captured in L241) > > Good catch. Capturing the current time in the listener will be more accurate > than doing it after the latch wait. Good catch indeed. Will test and report back accordingly. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1394#discussion_r1671923946
