[Webkit-unassigned] [Bug 83539] Web Inspector: ASSERT attempting to unbind null contentDocument

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Apr 9 19:56:04 PDT 2012


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





--- Comment #4 from Joseph Pecoraro <joepeck at webkit.org>  2012-04-09 19:56:04 PST ---
(In reply to comment #3)
> (From update of attachment 136374 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=136374&action=review
> 
> > Source/WebCore/inspector/InspectorDOMAgent.cpp:338
> > +            m_domListener->didRemoveDocument(contentDocument);
> 
> Is it safe to pass a null contenDocument to didRemoveDocument?

Yep, and I think its preferred since that does other work we might want:

    void InspectorCSSAgent::didRemoveDocument(Document* document)
    {
        if (document)
            m_documentToInspectorStyleSheet.remove(document);
        clearPseudoState(false);
    }

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



More information about the webkit-unassigned mailing list