RFR: 8185831: Psuedo selectors do not appear to work in Node.lookupAll() [v2]
John Hendrikx
jhendrikx at openjdk.org
Wed Oct 18 12:35:13 UTC 2023
On Wed, 18 Oct 2023 03:45:32 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.
>
> Sai Pradeep Dandem has updated the pull request incrementally with one additional commit since the last revision:
>
> 8185831: Fixed test failing issues and code review comments
This looks more reasonable now.
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1245#issuecomment-1768355060
More information about the openjfx-dev
mailing list