Integrated: 8185831: Pseudo selectors do not appear to work in Node.lookupAll()

Sai Pradeep Dandem duke at openjdk.org
Wed Nov 1 22:49:15 UTC 2023


On Tue, 19 Sep 2023 12:17:16 GMT, Sai Pradeep Dandem <duke at openjdk.org> wrote:

> **Issue:**
> Using pseudo classes in programmatic query using Node.lookupAll() or Node.lookup() gives unexpected results.
> 
> **Cause:**
> There is no check for checking the psuedo states matching in the applies() method of SimpleSelector.java. So checking for "applies()" alone is not sufficient in lookup() method.
> 
> **Fix:**
> Included an extra check for the psuedo states to match.

This pull request has now been integrated.

Changeset: bb06b64c
Author:    Dandem, Sai Pradeep <saipradeep.dandem at gmail.com>
Committer: Andy Goryachev <angorya at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/bb06b64c9e1cda9390a84704a75b530c1d1d4577
Stats:     185 lines in 2 files changed: 170 ins; 0 del; 15 mod

8185831: Pseudo selectors do not appear to work in Node.lookupAll()

Reviewed-by: kcr, angorya, jhendrikx

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

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


More information about the openjfx-dev mailing list