> 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 refreshed the contents of this pull request, and previous 
commits have been removed. The incremental views will show differences compared 
to the previous content of the PR. The pull request contains one new 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/a855c8f5..634793c9

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

  Stats: 3 lines in 1 file changed: 0 ins; 1 del; 2 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