When loading a MHTML document, make sure we set the base URL before
authorjcivelli@chromium.org <jcivelli@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 19 Nov 2011 08:45:01 +0000 (08:45 +0000)
committerjcivelli@chromium.org <jcivelli@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 19 Nov 2011 08:45:01 +0000 (08:45 +0000)
we notify the frame was committed (so the document has the right base
URL when the notification happens).
https://bugs.webkit.org/show_bug.cgi?id=72788

Reviewed by Adam Barth.

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

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

Source/WebCore/ChangeLog
Source/WebCore/loader/FrameLoader.cpp

index 7d0e477..d33e45b 100644 (file)
@@ -1,3 +1,15 @@
+2011-11-19  Jay Civelli  <jcivelli@chromium.org>
+
+        When loading a MHTML document, make sure we set the base URL before
+        we notify the frame was committed (so the document has the right base
+        URL when the notification happens). 
+        https://bugs.webkit.org/show_bug.cgi?id=72788
+
+        Reviewed by Adam Barth.
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::receivedFirstData):
+
 2011-11-19  Adam Barth  <abarth@webkit.org>
 
         Integrate Source/WTF with the Chromium build system
index 84ed4d2..b27b9b8 100644 (file)
@@ -574,9 +574,6 @@ void FrameLoader::receivedFirstData()
     activeDocumentLoader()->writer()->begin(workingURL, false);
     activeDocumentLoader()->writer()->setDocumentWasLoadedAsPartOfNavigation();
 
-    dispatchDidCommitLoad();
-    dispatchDidClearWindowObjectsInAllWorlds();
-
 #if ENABLE(MHTML)
     if (m_archive) {
         // The origin is the MHTML file, we need to set the base URL to the document encoded in the MHTML so
@@ -585,6 +582,9 @@ void FrameLoader::receivedFirstData()
     }
 #endif
 
+    dispatchDidCommitLoad();
+    dispatchDidClearWindowObjectsInAllWorlds();
+
     if (m_documentLoader) {
         StringWithDirection ptitle = m_documentLoader->title();
         // If we have a title let the WebView know about it.