107801 2013-01-24 01:03:51 -0800 The deleting Node should be detached. 2023-01-03 13:06:30 -0800 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 107640 1 morrita webkit-unassigned ahmad.saleem792 ap bfulgham rniwa oldest_to_newest 815214 0 morrita 2013-01-24 01:03:51 -0800 A tracking bug which comes from Bug 107640. In some case, Node::attached() is true even the node is deleted. We should be able to put ASSERT(!attached()) in ContainerNode::removeDetachedNode() but that makes some tests fail. It should be fixed to discipline the Node lifecycle. 1922692 1 ahmad.saleem792 2023-01-03 13:04:42 -0800 Blink fixed this in following commit - https://chromium.googlesource.com/chromium/blink/+/7977c3ddb18c061e3fe161c7823c78416693580d%5E%21/#F0 Rather than asserting "attached", they did "connectedSubframeCount". WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/dom/ContainerNode.cpp#321 1922693 2 ahmad.saleem792 2023-01-03 13:06:30 -0800 (In reply to Ahmad Saleem from comment #1) > Blink fixed this in following commit - > https://chromium.googlesource.com/chromium/blink/+/ > 7977c3ddb18c061e3fe161c7823c78416693580d%5E%21/#F0 > > Rather than asserting "attached", they did "connectedSubframeCount". > > WebKit Source - > https://searchfox.org/wubkat/source/Source/WebCore/dom/ContainerNode.cpp#321 Ignore me - they have another ASSERT with 'needsAttach'.