[Webkit-unassigned] [Bug 60419] Assertion failure trying to remove a nodelist from the nodelist cache when no nodelist is present.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=60419


Oliver Hunt <oliver at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |NeedsRadar




--- Comment #1 from Oliver Hunt <oliver at apple.com>  2011-05-06 17:19:27 PST ---
>From code inspection i saw

#0  WebCore::Node::setTreeScopeRecursively (this=0x13238ebc0, newTreeScope=0x10a69a000, includeRoot=true) at /Volumes/Data/git/WebKit/OpenSource/Source/WebCore/dom/Node.cpp:508
508                newTreeScope->addNodeListCache();
(gdb) list 
503                node->ensureRareData()->setTreeScope(newTreeScope);
504    
505            if (hasRareData() && rareData()->nodeLists()) {
506                if (currentTreeScope)
507                    currentTreeScope->removeNodeListCache();
508                newTreeScope->addNodeListCache();
509            }


should this be:
505            if (currentTreeScope->hasRareData() && currentTreeScope->rareData()->nodeLists()) {
506                if (currentTreeScope)
507                    currentTreeScope->removeNodeListCache();
508                newTreeScope->addNodeListCache();
509            }

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.