[Qt][WK2] fast/loader/create-frame-in-DOMContentLoaded.html crashes
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 May 2012 15:15:51 +0000 (15:15 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 May 2012 15:15:51 +0000 (15:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=83578

Source/WebKit2:

Retrieve and save page ID only for a valid page in WebFrameNetworkingContext.
The page will be null if the frame was detached from the page.

Patch by Dinu Jacob <dinu.jacob@nokia.com> on 2012-05-15
Reviewed by Simon Hausmann.

* WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.cpp:
(WebCore::WebFrameNetworkingContext::WebFrameNetworkingContext):

LayoutTests:

Unskip passing test.

Patch by Dinu Jacob <dinu.jacob@nokia.com> on 2012-05-15
Reviewed by Simon Hausmann.

* platform/qt-5.0-wk2/Skipped:

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

LayoutTests/ChangeLog
LayoutTests/platform/qt-5.0-wk2/Skipped
Source/WebKit2/ChangeLog
Source/WebKit2/WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.cpp

index 0d68c3e..f367d4e 100644 (file)
@@ -1,3 +1,14 @@
+2012-05-15  Dinu Jacob  <dinu.jacob@nokia.com>
+
+        [Qt][WK2] fast/loader/create-frame-in-DOMContentLoaded.html crashes
+        https://bugs.webkit.org/show_bug.cgi?id=83578
+
+        Unskip passing test.
+
+        Reviewed by Simon Hausmann.
+
+        * platform/qt-5.0-wk2/Skipped:
+
 2012-05-15  Vsevolod Vlasov  <vsevik@chromium.org>
 
         Web Inspector: Refactor SnippetsModel: extract SnippetStorage, make SnippetScriptMapping delegate calls to ScriptSnippetModel and provide public interface on ScriptSnippetModel in terms of uiSourceCodes
index d368a26..7c6c078 100644 (file)
@@ -574,10 +574,6 @@ plugins
 # https://bugs.webkit.org/show_bug.cgi?id=83575
 fast/frames/iframe-window-focus.html
 
-# [Qt][WK2] fast/loader/create-frame-in-DOMContentLoaded.html crashes
-# https://bugs.webkit.org/show_bug.cgi?id=83578
-fast/loader/create-frame-in-DOMContentLoaded.html
-
 # Missing layoutTestController.numberOfPendingGeolocationPermissionRequests() implementation
 fast/dom/Geolocation/page-reload-cancel-permission-requests.html
 
index 3230964..bc67b3b 100644 (file)
@@ -1,3 +1,16 @@
+2012-05-15  Dinu Jacob  <dinu.jacob@nokia.com>
+
+        [Qt][WK2] fast/loader/create-frame-in-DOMContentLoaded.html crashes
+        https://bugs.webkit.org/show_bug.cgi?id=83578
+
+        Retrieve and save page ID only for a valid page in WebFrameNetworkingContext.
+        The page will be null if the frame was detached from the page.
+
+        Reviewed by Simon Hausmann.
+
+        * WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.cpp:
+        (WebCore::WebFrameNetworkingContext::WebFrameNetworkingContext):
+
 2012-05-15  Allan Sandfeld Jensen  <allan.jensen@nokia.com>
 
         [Qt][WK2] Fix scrolling in touch mode
index 070d479..387304f 100644 (file)
@@ -31,11 +31,13 @@ namespace WebCore {
 
 WebFrameNetworkingContext::WebFrameNetworkingContext(WebKit::WebFrame* frame)
     : FrameNetworkingContext(frame->coreFrame())
-    , m_originatingObject(adoptPtr(new QObject))
     , m_mimeSniffingEnabled(true)
 {
-    // The page ID is needed later for HTTP authentication and SSL errors.
-    m_originatingObject->setProperty("pageID", qulonglong(frame->page()->pageID()));
+    // Save the page ID for a valid page as it is needed later for HTTP authentication and SSL errors.
+    if (frame->page()) {
+        m_originatingObject = adoptPtr(new QObject);
+        m_originatingObject->setProperty("pageID", qulonglong(frame->page()->pageID()));
+    }
 }
 
 WebFrameNetworkingContext::~WebFrameNetworkingContext()