Make a bunch of DocumentLoader functions private
authorjaphet@chromium.org <japhet@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 1 Apr 2013 16:54:16 +0000 (16:54 +0000)
committerjaphet@chromium.org <japhet@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 1 Apr 2013 16:54:16 +0000 (16:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=113601

Reviewed by Alexey Proskuryakov.

No new tests, cleanup only.

* loader/DocumentLoader.cpp:
* loader/DocumentLoader.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@147336 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/loader/DocumentLoader.cpp
Source/WebCore/loader/DocumentLoader.h

index 68051f8..91dd962 100644 (file)
@@ -1,3 +1,15 @@
+2013-04-01  Nate Chapin  <japhet@chromium.org>
+
+        Make a bunch of DocumentLoader functions private
+        https://bugs.webkit.org/show_bug.cgi?id=113601
+
+        Reviewed by Alexey Proskuryakov.
+
+        No new tests, cleanup only.
+
+        * loader/DocumentLoader.cpp:
+        * loader/DocumentLoader.h:
+
 2013-04-01  Kangil Han  <kangil.han@samsung.com>
 
         Minor code cleanup by removing duplicated null checks
 2013-04-01  Kangil Han  <kangil.han@samsung.com>
 
         Minor code cleanup by removing duplicated null checks
index d1190b9..61cff07 100644 (file)
@@ -452,7 +452,7 @@ void DocumentLoader::startDataLoadTimer()
 
 void DocumentLoader::handleSubstituteDataLoadSoon()
 {
 
 void DocumentLoader::handleSubstituteDataLoadSoon()
 {
-    if (deferMainResourceDataLoad())
+    if (m_deferMainResourceDataLoad)
         startDataLoadTimer();
     else
         handleSubstituteDataLoadNow(0);
         startDataLoadTimer();
     else
         handleSubstituteDataLoadNow(0);
@@ -612,7 +612,7 @@ void DocumentLoader::responseReceived(CachedResource* resource, const ResourceRe
         m_isLoadingMultipartContent = true;
     }
 
         m_isLoadingMultipartContent = true;
     }
 
-    setResponse(response);
+    m_response = response;
 
     if (m_identifierForLoadWithoutResourceLoader)
         frameLoader()->notifier()->dispatchDidReceiveResponse(this, m_identifierForLoadWithoutResourceLoader, m_response, 0);
 
     if (m_identifierForLoadWithoutResourceLoader)
         frameLoader()->notifier()->dispatchDidReceiveResponse(this, m_identifierForLoadWithoutResourceLoader, m_response, 0);
@@ -845,7 +845,7 @@ void DocumentLoader::dataReceived(CachedResource* resource, const char* data, in
 #if USE(CFNETWORK) || PLATFORM(MAC)
     // Workaround for <rdar://problem/6060782>
     if (m_response.isNull())
 #if USE(CFNETWORK) || PLATFORM(MAC)
     // Workaround for <rdar://problem/6060782>
     if (m_response.isNull())
-        setResponse(ResourceResponse(KURL(), "text/html", 0, String(), String()));
+        m_response = ResourceResponse(KURL(), "text/html", 0, String(), String());
 #endif
 
     // There is a bug in CFNetwork where callbacks can be dispatched even when loads are deferred.
 #endif
 
     // There is a bug in CFNetwork where callbacks can be dispatched even when loads are deferred.
@@ -1270,7 +1270,7 @@ KURL DocumentLoader::documentURL() const
     if (url.isEmpty())
         url = requestURL();
     if (url.isEmpty())
     if (url.isEmpty())
         url = requestURL();
     if (url.isEmpty())
-        url = responseURL();
+        url = m_response.url();
     return url;
 }
 
     return url;
 }
 
@@ -1363,7 +1363,7 @@ bool DocumentLoader::maybeLoadEmpty()
     if (m_request.url().isEmpty() && !frameLoader()->stateMachine()->creatingInitialEmptyDocument())
         m_request.setURL(blankURL());
     String mimeType = shouldLoadEmpty ? "text/html" : frameLoader()->client()->generatedMIMETypeForURLScheme(m_request.url().protocol());
     if (m_request.url().isEmpty() && !frameLoader()->stateMachine()->creatingInitialEmptyDocument())
         m_request.setURL(blankURL());
     String mimeType = shouldLoadEmpty ? "text/html" : frameLoader()->client()->generatedMIMETypeForURLScheme(m_request.url().protocol());
-    setResponse(ResourceResponse(m_request.url(), mimeType, 0, String(), String()));
+    m_response = ResourceResponse(m_request.url(), mimeType, 0, String(), String());
     finishedLoading(monotonicallyIncreasingTime());
     return true;
 }
     finishedLoading(monotonicallyIncreasingTime());
     return true;
 }
index c4b99af..9418ecf 100644 (file)
@@ -96,14 +96,12 @@ namespace WebCore {
         PassRefPtr<ResourceBuffer> mainResourceData() const;
         
         DocumentWriter* writer() const { return &m_writer; }
         PassRefPtr<ResourceBuffer> mainResourceData() const;
         
         DocumentWriter* writer() const { return &m_writer; }
-        Document* document() const;
 
         const ResourceRequest& originalRequest() const;
         const ResourceRequest& originalRequestCopy() const;
 
         const ResourceRequest& request() const;
         ResourceRequest& request();
 
         const ResourceRequest& originalRequest() const;
         const ResourceRequest& originalRequestCopy() const;
 
         const ResourceRequest& request() const;
         ResourceRequest& request();
-        void setRequest(const ResourceRequest&);
 
         CachedResourceLoader* cachedResourceLoader() const { return m_cachedResourceLoader.get(); }
 
 
         CachedResourceLoader* cachedResourceLoader() const { return m_cachedResourceLoader.get(); }
 
@@ -113,9 +111,6 @@ namespace WebCore {
         const KURL& url() const;
         const KURL& unreachableURL() const;
 
         const KURL& url() const;
         const KURL& unreachableURL() const;
 
-        // The URL of the document resulting from this DocumentLoader.
-        KURL documentURL() const;
-
         const KURL& originalURL() const;
         const KURL& requestURL() const;
         const KURL& responseURL() const;
         const KURL& originalURL() const;
         const KURL& requestURL() const;
         const KURL& responseURL() const;
@@ -127,13 +122,8 @@ namespace WebCore {
         void setCommitted(bool committed) { m_committed = committed; }
         bool isCommitted() const { return m_committed; }
         bool isLoading() const;
         void setCommitted(bool committed) { m_committed = committed; }
         bool isCommitted() const { return m_committed; }
         bool isLoading() const;
-        void receivedData(const char*, int);
-        void setupForReplace();
-        void finishedLoading(double finishTime);
         const ResourceResponse& response() const { return m_response; }
         const ResourceError& mainDocumentError() const { return m_mainDocumentError; }
         const ResourceResponse& response() const { return m_response; }
         const ResourceError& mainDocumentError() const { return m_mainDocumentError; }
-        void mainReceivedError(const ResourceError&);
-        void setResponse(const ResourceResponse& response) { m_response = response; }
         bool isClientRedirect() const { return m_isClientRedirect; }
         void setIsClientRedirect(bool isClientRedirect) { m_isClientRedirect = isClientRedirect; }
         void handledOnloadEvents();
         bool isClientRedirect() const { return m_isClientRedirect; }
         void setIsClientRedirect(bool isClientRedirect) { m_isClientRedirect = isClientRedirect; }
         void handledOnloadEvents();
@@ -230,10 +220,7 @@ namespace WebCore {
 
         void subresourceLoaderFinishedLoadingOnePart(ResourceLoader*);
 
 
         void subresourceLoaderFinishedLoadingOnePart(ResourceLoader*);
 
-        void maybeFinishLoadingMultipartContent();
-
         void setDeferMainResourceDataLoad(bool defer) { m_deferMainResourceDataLoad = defer; }
         void setDeferMainResourceDataLoad(bool defer) { m_deferMainResourceDataLoad = defer; }
-        bool deferMainResourceDataLoad() const { return m_deferMainResourceDataLoad; }
         
         void didTellClientAboutLoad(const String& url)
         { 
         
         void didTellClientAboutLoad(const String& url)
         { 
@@ -253,8 +240,6 @@ namespace WebCore {
         DocumentLoadTiming* timing() { return &m_documentLoadTiming; }
         void resetTiming() { m_documentLoadTiming = DocumentLoadTiming(); }
 
         DocumentLoadTiming* timing() { return &m_documentLoadTiming; }
         void resetTiming() { m_documentLoadTiming = DocumentLoadTiming(); }
 
-        void responseReceived(const ResourceResponse&);
-
         // The WebKit layer calls this function when it's ready for the data to
         // actually be added to the document.
         void commitData(const char* bytes, size_t length);
         // The WebKit layer calls this function when it's ready for the data to
         // actually be added to the document.
         void commitData(const char* bytes, size_t length);
@@ -270,10 +255,20 @@ namespace WebCore {
         bool m_deferMainResourceDataLoad;
 
     private:
         bool m_deferMainResourceDataLoad;
 
     private:
+
+        // The URL of the document resulting from this DocumentLoader.
+        KURL documentURL() const;
+        Document* document() const;
+
+        void setRequest(const ResourceRequest&);
+
         void commitIfReady();
         void setMainDocumentError(const ResourceError&);
         void commitLoad(const char*, int);
         void clearMainResourceLoader();
         void commitIfReady();
         void setMainDocumentError(const ResourceError&);
         void commitLoad(const char*, int);
         void clearMainResourceLoader();
+
+        void setupForReplace();
+        void maybeFinishLoadingMultipartContent();
         
         bool maybeCreateArchive();
 #if ENABLE(WEB_ARCHIVE) || ENABLE(MHTML)
         
         bool maybeCreateArchive();
 #if ENABLE(WEB_ARCHIVE) || ENABLE(MHTML)
@@ -281,6 +276,8 @@ namespace WebCore {
 #endif
 
         void willSendRequest(ResourceRequest&, const ResourceResponse&);
 #endif
 
         void willSendRequest(ResourceRequest&, const ResourceResponse&);
+        void finishedLoading(double finishTime);
+        void mainReceivedError(const ResourceError&);
         virtual void redirectReceived(CachedResource*, ResourceRequest&, const ResourceResponse&) OVERRIDE;
         virtual void responseReceived(CachedResource*, const ResourceResponse&) OVERRIDE;
         virtual void dataReceived(CachedResource*, const char* data, int length) OVERRIDE;
         virtual void redirectReceived(CachedResource*, ResourceRequest&, const ResourceResponse&) OVERRIDE;
         virtual void responseReceived(CachedResource*, const ResourceResponse&) OVERRIDE;
         virtual void dataReceived(CachedResource*, const char* data, int length) OVERRIDE;