Changeset 67198 in webkit


Ignore:
Timestamp:
Sep 10, 2010 7:30:26 AM (14 years ago)
Author:
abarth@webkit.org
Message:

2010-09-10 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Inline FrameLoader::committedLoad into its one caller
https://bugs.webkit.org/show_bug.cgi?id=45481

This saves one round-trip through the FrameLoader when loading
documents.

  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::commitLoad):
  • loader/FrameLoader.cpp:
  • loader/FrameLoader.h:
Location:
trunk/WebCore
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebCore/ChangeLog

    r67197 r67198  
     12010-09-10  Adam Barth  <abarth@webkit.org>
     2
     3        Reviewed by Eric Seidel.
     4
     5        Inline FrameLoader::committedLoad into its one caller
     6        https://bugs.webkit.org/show_bug.cgi?id=45481
     7
     8        This saves one round-trip through the FrameLoader when loading
     9        documents.
     10
     11        * loader/DocumentLoader.cpp:
     12        (WebCore::DocumentLoader::commitLoad):
     13        * loader/FrameLoader.cpp:
     14        * loader/FrameLoader.h:
     15
    1162010-09-10  Patrick Gansterer  <paroga@paroga.com>
    217
  • trunk/WebCore/loader/DocumentLoader.cpp

    r66963 r67198  
    4040#include "Frame.h"
    4141#include "FrameLoader.h"
     42#include "FrameLoaderClient.h"
    4243#include "FrameTree.h"
    4344#include "HistoryItem.h"
     
    278279
    279280    commitIfReady();
    280     if (FrameLoader* frameLoader = DocumentLoader::frameLoader())
    281         frameLoader->committedLoad(this, data, length);
     281    FrameLoader* frameLoader = DocumentLoader::frameLoader();
     282    if (!frameLoader)
     283        return;
     284    if (ArchiveFactory::isArchiveMimeType(response().mimeType()))
     285        return;
     286    frameLoader->client()->committedLoad(this, data, length);
    282287}
    283288
  • trunk/WebCore/loader/FrameLoader.cpp

    r66963 r67198  
    27092709}
    27102710
    2711 void FrameLoader::committedLoad(DocumentLoader* loader, const char* data, int length)
    2712 {
    2713     if (ArchiveFactory::isArchiveMimeType(loader->response().mimeType()))
    2714         return;
    2715     m_client->committedLoad(loader, data, length);
    2716 }
    2717 
    27182711void FrameLoader::loadPostRequest(const ResourceRequest& inRequest, const String& referrer, const String& frameName, bool lockHistory, FrameLoadType loadType, PassRefPtr<Event> event, PassRefPtr<FormState> prpFormState)
    27192712{
  • trunk/WebCore/loader/FrameLoader.h

    r66794 r67198  
    180180    void didReceiveServerRedirectForProvisionalLoadForFrame();
    181181    void finishedLoadingDocument(DocumentLoader*);
    182     void committedLoad(DocumentLoader*, const char*, int);
    183182    bool isReplacing() const;
    184183    void setReplacing();
Note: See TracChangeset for help on using the changeset viewer.