Title: [100852] trunk/Source/WebCore
Revision
100852
Author
[email protected]
Date
2011-11-19 00:45:01 -0800 (Sat, 19 Nov 2011)

Log Message

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):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (100851 => 100852)


--- trunk/Source/WebCore/ChangeLog	2011-11-19 08:25:10 UTC (rev 100851)
+++ trunk/Source/WebCore/ChangeLog	2011-11-19 08:45:01 UTC (rev 100852)
@@ -1,3 +1,15 @@
+2011-11-19  Jay Civelli  <[email protected]>
+
+        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  <[email protected]>
 
         Integrate Source/WTF with the Chromium build system

Modified: trunk/Source/WebCore/loader/FrameLoader.cpp (100851 => 100852)


--- trunk/Source/WebCore/loader/FrameLoader.cpp	2011-11-19 08:25:10 UTC (rev 100851)
+++ trunk/Source/WebCore/loader/FrameLoader.cpp	2011-11-19 08:45:01 UTC (rev 100852)
@@ -574,9 +574,6 @@
     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 @@
     }
 #endif
 
+    dispatchDidCommitLoad();
+    dispatchDidClearWindowObjectsInAllWorlds();
+
     if (m_documentLoader) {
         StringWithDirection ptitle = m_documentLoader->title();
         // If we have a title let the WebView know about it.
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to