Changeset 67224 in webkit

Timestamp:

Sep 10, 2010 1:01:53 PM (14 years ago)

Author:

abarth@webkit.org

Message:

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

Reviewed by Darin Adler.

Remove useless FrameLoader method
​https://bugs.webkit.org/show_bug.cgi?id=45492

This method has one caller, and that caller already has a more direct
way of accessing the correct DocumentLoader.

• loader/FrameLoader.cpp:
• loader/FrameLoader.h:
• loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::addData):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/FrameLoader.cpp (modified) (1 diff)
• loader/FrameLoader.h (modified) (1 diff)
• loader/MainResourceLoader.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-09-10  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Darin Adler.
+
+        Remove useless FrameLoader method
+        https://bugs.webkit.org/show_bug.cgi?id=45492
+
+        This method has one caller, and that caller already has a more direct
+        way of accessing the correct DocumentLoader.
+
+        * loader/FrameLoader.cpp:
+        * loader/FrameLoader.h:
+        * loader/MainResourceLoader.cpp:
+        (WebCore::MainResourceLoader::addData):
+
 2010-09-10  Adam Barth  <abarth@webkit.org>
 

trunk/WebCore/loader/FrameLoader.cpp

@@ -1494 +1494 @@
 }
 
-void FrameLoader::receivedData(const char* data, int length)
-{
-    activeDocumentLoader()->receivedData(data, length);
-}
-
 bool FrameLoader::willLoadMediaElementURL(KURL& url)
 {

trunk/WebCore/loader/FrameLoader.h

@@ -154 +154 @@
     const ResourceRequest& initialRequest() const;
     void receivedMainResourceError(const ResourceError&, bool isComplete);
-    void receivedData(const char*, int);
 
     bool willLoadMediaElementURL(KURL&);

trunk/WebCore/loader/MainResourceLoader.cpp

@@ -147 +147 @@
 {
     ResourceLoader::addData(data, length, allAtOnce);
-    frameLoader()->receivedData(data, length);
+    documentLoader()->receivedData(data, length);
 }