Changeset 51676 in webkit


Ignore:
Timestamp:
Dec 3, 2009 9:28:33 PM (14 years ago)
Author:
abarth@webkit.org
Message:

2009-12-03 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

The code should say that Document owns DocLoader
https://bugs.webkit.org/show_bug.cgi?id=32143

It's the truth.

  • dom/Document.cpp: (WebCore::Document::~Document):
  • dom/Document.h: (WebCore::Document::docLoader):
Location:
trunk/WebCore
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebCore/ChangeLog

    r51670 r51676  
     12009-12-03  Adam Barth  <abarth@webkit.org>
     2
     3        Reviewed by Eric Seidel.
     4
     5        The code should say that Document owns DocLoader
     6        https://bugs.webkit.org/show_bug.cgi?id=32143
     7
     8        It's the truth.
     9
     10        * dom/Document.cpp:
     11        (WebCore::Document::~Document):
     12        * dom/Document.h:
     13        (WebCore::Document::docLoader):
     14
    1152009-12-03  Chris Fleizach  <cfleizach@apple.com>
    216
  • trunk/WebCore/dom/Document.cpp

    r51644 r51676  
    486486    m_document = 0;
    487487    delete m_styleSelector;
    488     delete m_docLoader;
    489    
     488    m_docLoader.clear();
     489
    490490    if (m_renderArena) {
    491491        delete m_renderArena;
  • trunk/WebCore/dom/Document.h

    r51644 r51676  
    454454    void updateLayoutIgnorePendingStylesheets();
    455455    static void updateStyleForAllDocuments(); // FIXME: Try to reduce the # of calls to this function.
    456     DocLoader* docLoader() { return m_docLoader; }
     456    DocLoader* docLoader() { return m_docLoader.get(); }
    457457
    458458    virtual void attach();
     
    961961
    962962    Frame* m_frame;
    963     DocLoader* m_docLoader;
     963    OwnPtr<DocLoader> m_docLoader;
    964964    Tokenizer* m_tokenizer;
    965965    bool m_wellFormed;
Note: See TracChangeset for help on using the changeset viewer.