> Tests for the fix made in https://github.com/openjdk/jfx/pull/2171.
> 
> I figured out that `ScrollEventTest` already has a way to test `Scene` 
> `ScrollEvent`s.
> This is done by invoking the `ScenePeerListener.scrollEvent(..)` method in 
> the tests.
> 
> This is the very same way the `GlassViewEventHandler` will do in a normal 
> application (which is usually invoked from 
> `com.sun.glass.ui.View.notifyScroll(..)`).
> 
> cc @beldenfox 
> This tests the picked `Node` and the `Scene`, if there is no `Node` to pick. 
> ~Let me know what you think, I will create a ticket if this is fine!~ Done.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Marius Hanl has updated the pull request incrementally with one additional 
commit since the last revision:

  Improve assertion messages

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/2194/files
  - new: https://git.openjdk.org/jfx/pull/2194/files/f2c1f8c3..a855c8f5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=2194&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2194&range=00-01

  Stats: 17 lines in 1 file changed: 1 ins; 0 del; 16 mod
  Patch: https://git.openjdk.org/jfx/pull/2194.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/2194/head:pull/2194

PR: https://git.openjdk.org/jfx/pull/2194

Reply via email to