Bug 179224 - MutationObserver callback should be kept alive as long as its observed nodes are alive
Summary: MutationObserver callback should be kept alive as long as its observed nodes ...
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-11-02 19:18 PDT by Ryosuke Niwa
Modified: 2018-10-02 22:43 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2017-11-02 19:18:07 PDT
We don't keep it alive because we use a weak reference in JSMutationCallback :(
Comment 1 Radar WebKit Bug Importer 2017-11-03 13:09:07 PDT
<rdar://problem/35341598>
Comment 2 Ryosuke Niwa 2018-10-02 22:43:55 PDT
This already works today because JSMutationObserverOwner::isReachableFromOpaqueRoots checks every node being observed, and JSMutationObserver::visitAdditionalChildren visits the callback.