nephele
/
haikuwebkit
1
0
Fork 0
Code Issues Releases Activity
haikuwebkit/LayoutTests/intersection-observer/intersection-observer-keeps...

24 lines
1.0 KiB
Plaintext
Raw Permalink Normal View History

Source/WebCore: REGRESSION(r279800): IntersectionObserver may never get a delivery of an observation if the element has been unobserved and is disconnected https://bugs.webkit.org/show_bug.cgi?id=228679 Reviewed by Chris Dumez. The bug was caused by the elements of the pending entries not being considered as opaque roots of the intersection observer. If the intersection observer is no longer observing those elements, the JS wrapper of the intersection observer may be collected erroneously. This patch addresses this issue by explicitly treating the elements of pending entries as opaque roots of the intersection observer in IntersectionObserver::isReachableFromOpaqueRoots. Test: intersection-observer/intersection-observer-keeps-element-of-queued-entry-alive.html * page/IntersectionObserver.cpp: (WebCore::IntersectionObserver::isReachableFromOpaqueRoots const): LayoutTests: REGRESSION(r279800): IntersectionObserver may never get a delivery of an observation if the element has been unobserved and is disconnected https://bugs.webkit.org/show_bug.cgi?id=228679 Reviewed by Chris Dumez. Added a regression test. * intersection-observer/intersection-observer-keeps-element-of-queued-entry-alive-expected.txt: Added. * intersection-observer/intersection-observer-keeps-element-of-queued-entry-alive.html: Added. Canonical link: https://commits.webkit.org/240176@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@280549 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-08-02 19:14:48 +00:00
This tests observing an element with an IntersectionObserver and removing the element from the document while it is queued for delivery.
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive
PASS - IntersectionObserver wrapper is alive