2008-11-04 Darin Fisher <darin@chromium.org>
authordarin@chromium.org <darin@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 4 Nov 2008 19:40:56 +0000 (19:40 +0000)
committerdarin@chromium.org <darin@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 4 Nov 2008 19:40:56 +0000 (19:40 +0000)
        Reviewed by Anders Carlsson.

        No need to clobber all ResourceRequest fields in FrameLoader::reload()
        https://bugs.webkit.org/show_bug.cgi?id=21949

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::reload):

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

WebCore/ChangeLog
WebCore/loader/FrameLoader.cpp

index 20597f28541175135216a379d2913af2457e148f..b005318d5e6ea1df2cf4d591d7118003ee0eebe3 100644 (file)
@@ -1,3 +1,13 @@
+2008-11-04  Darin Fisher  <darin@chromium.org>
+        Reviewed by Anders Carlsson.
+        No need to clobber all ResourceRequest fields in FrameLoader::reload()
+        https://bugs.webkit.org/show_bug.cgi?id=21949
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::reload):
+
 2008-11-04  Darin Adler  <darin@apple.com>
 
         Reviewed by Tim Hatcher.
index 8516adfe92f7f618af9d6799eff6e4a8ec9e91d5..e8699e52ad65910e828100a3fd97fed2e3b136f7 100644 (file)
@@ -2437,17 +2437,17 @@ void FrameLoader::reload()
     if (!m_documentLoader)
         return;
 
-    ResourceRequest& initialRequest = m_documentLoader->request();
-    
     // If a window is created by javascript, its main frame can have an empty but non-nil URL.
     // Reloading in this case will lose the current contents (see 4151001).
-    if (initialRequest.url().isEmpty())
+    if (m_documentLoader->request().url().isEmpty())
         return;
 
+    ResourceRequest initialRequest = m_documentLoader->request();
+
     // Replace error-page URL with the URL we were trying to reach.
     KURL unreachableURL = m_documentLoader->unreachableURL();
     if (!unreachableURL.isEmpty())
-        initialRequest = ResourceRequest(unreachableURL);
+        initialRequest.setURL(unreachableURL);
     
     // Create a new document loader for the reload, this will become m_documentLoader eventually,
     // but first it has to be the "policy" document loader, and then the "provisional" document loader.