| Summary: | REGRESSION: (r256232) [ Mac wk1 Release ] resize-observer/delete-observers-in-callbacks.html is flaky failing. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Jason Lawrence <Lawrence.j> |
| Component: | New Bugs | Assignee: | Chris Dumez <cdumez> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | cdumez, jacob_uphoff, simon.fraser, webkit-bot-watchers-bugzilla, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Mac | ||
| OS: | macOS 10.14 | ||
|
Description
Jason Lawrence
2020-02-17 10:29:07 PST
I have marked this test as failing while this issue is investigated. https://trac.webkit.org/changeset/256744/webkit Reproducible by running the test in iterations. Ran this command: 'run-webkit-tests --root release-256766 resize-observer/delete-observers-in-callbacks.html -1 -f --iterations 5000' and failed 8 times. Looks like it might not be a recent regression, I have run the test back to commit 256422 and am seeing the failure there. It is curious because the failures will happen pretty early on in the iteration runs then pass for the rest of them. Results: [8/5002] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) [8/5002] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) [10/5002] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) [11/5002] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) [12/5002] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) [18/5002] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) [22/5002] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) [24/5002] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) No handlers could be found for logger "webkitpy.common.interrupt_debugging" Retrying 1 unexpected failure ... Running 1 DumpRenderTree. [1/1] resize-observer/delete-observers-in-callbacks.html failed unexpectedly (text diff) 4994 tests ran as expected, 8 didn't: I am able to reproduce this issue with r256232, but not with r256231 using this command: run-webkit-tests resize-observer/delete-observers-in-callbacks.html --force -f --iterations 5000 --exit-after-n-failures 5 -1 Looks like ResizeObserver is enabled by a setting: EnabledBySetting=ResizeObserver The test also uses the following header: <!DOCTYPE html><!-- webkit-test-runner [ experimental:ResizeObserverEnabled=true ] --> (In reply to Chris Dumez from comment #7) > The test also uses the following header: > <!DOCTYPE html><!-- webkit-test-runner [ > experimental:ResizeObserverEnabled=true ] --> The issue is that some settings can be set either via internals JS API or via <!-- webkit-test-runner -->. Depending on which order you do these things, you'll end up with a different results. Also, to make things more complicated, once the window object has been created, changes to the ResizeObserverEnabled setting are no longer reflected on that Window object. It is important that ResizeObserverEnabled setting be enabled *before* we construct the window. Reverted guilty patch in <https://trac.webkit.org/changeset/258438>. Committed r258440: <https://trac.webkit.org/changeset/258440> |