260556 2023-08-22 16:22:45 -0700 fast/dom/selectorAPI/caseID.html is a constant text failure (when repro steps are followed). 2023-08-24 11:45:20 -0700 1 1 1 Unclassified WebKit DOM WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=254732 InRadar P2 Normal --- 1 ben_schwartz cdumez annevk cdumez koivisto webkit-bot-watchers-bugzilla webkit-bug-importer oldest_to_newest 1972775 0 ben_schwartz 2023-08-22 16:22:45 -0700 fast/dom/selectorAPI/caseID.html This test is a flaky failure on macOS and iOS. HISTORY (link throwing 403 error): https://results.webkit.org/?suite=layout-tests&test=fast%2Fdom%2FSelectorAPI%2FcaseID.html TEXT DIFF: PASS document.querySelector('#lower1').textContent is 'lower 1' -PASS document.querySelector('#LOWER2').textContent is 'lower 2' +FAIL document.querySelector('#LOWER2').textContent should be lower 2. Threw exception TypeError: null is not an object (evaluating 'document.querySelector('#LOWER2').textContent') PASS document.querySelector('#UPPER1').textContent is 'UPPER 1' PASS document.querySelector('#upper2').textContent is 'UPPER 2' PASS document.getElementById('lower1').matches('#lower1') is true -PASS document.getElementById('lower2').matches('#LOWER2') is true +FAIL document.getElementById('lower2').matches('#LOWER2') should be true. Was false. PASS document.getElementById('UPPER1').matches('#UPPER1') is true PASS document.getElementById('UPPER2').matches('#upper2') is true PASS document.getElementById('lower1').webkitMatchesSelector('#lower1') is true -PASS document.getElementById('lower2').webkitMatchesSelector('#LOWER2') is true +FAIL document.getElementById('lower2').webkitMatchesSelector('#LOWER2') should be true. Was false. PASS document.getElementById('UPPER1').webkitMatchesSelector('#UPPER1') is true PASS document.getElementById('UPPER2').webkitMatchesSelector('#upper2') is true PASS successfullyParsed is true DIFF URL: https://build.webkit.org/results/Apple-Ventura-Release-AppleSilicon-WK2-Tests/267146%40main%20(4573)/fast/dom/SelectorAPI/caseID-pretty-diff.html REPRODUCTION: I was unable to reproduce this bug on macOS Ventura 13.5 (22G74/arm64). I will do more research. 1972776 1 webkit-bug-importer 2023-08-22 16:23:05 -0700 <rdar://problem/114287946> 1972798 2 ben_schwartz 2023-08-22 17:29:54 -0700 REPRODUCTION: It appears that the failure of this test depends on these three tests being run before: fast/dom/HTMLMeterElement/meter-clone.html fast/dom/SelectorAPI/attrname-case-sensitive.xhtml fast/dom/SelectorAPI/caseID-strict.html When running these tests in sequence prior to the reported test, I was able to reproduce this bug on macOS Ventura 13.5 (22G74/arm64) with at a constant rate using the following command: run-webkit-tests --verbose --clobber-old-results --debug --child-processes 1 fast/dom/HTMLMeterElement/meter-clone.html fast/dom/SelectorAPI/attrname-case-sensitive.xhtml fast/dom/SelectorAPI/caseID-strict.html fast/dom/SelectorAPI/caseID.html 1972800 3 ews-feeder 2023-08-22 17:48:26 -0700 Test gardening commit 267166@main (f11bf4050d7b): <https://commits.webkit.org/267166@main> Reviewed commits have been landed. Closing PR #16957 and removing active labels. 1972924 4 annevk 2023-08-23 00:45:08 -0700 Antti, you might want to take a look at this. 1973118 5 cdumez 2023-08-23 16:05:51 -0700 This is caused by the SelectorQueryCache. Disabling that cache makes the issue go away. Likely a regression from when Antti made the SelectorQueryCache be shared across documents. 1973120 6 cdumez 2023-08-23 16:07:13 -0700 (In reply to Chris Dumez from comment #5) > This is caused by the SelectorQueryCache. Disabling that cache makes the > issue go away. Likely a regression from when Antti made the > SelectorQueryCache be shared across documents. I was referring to Bug 254732. 1973283 7 cdumez 2023-08-24 08:56:26 -0700 Pull request: https://github.com/WebKit/WebKit/pull/17016 1973321 8 ews-feeder 2023-08-24 11:45:17 -0700 Committed 267237@main (727dbe30fe5f): <https://commits.webkit.org/267237@main> Reviewed commits have been landed. Closing PR #17016 and removing active labels.