Attachment 343585 Details for Bug 98345 – Patch

[patch] Patch

bug-98345-20180625225821.patch (text/plain), 3.28 KB, created by Fujii Hironori on 2018-06-25 22:58:55 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Fujii Hironori

Created: 2018-06-25 22:58:55 PDT

Size: 3.28 KB

patch

obsolete

>Subversion Revision: 233142
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index c6a2cfa5193f3b497c90164ba5f56d96026c95f6..6583174426900127d539e33f97d9febe57962130 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,19 @@
>+2018-06-25  Fujii Hironori  <Hironori.Fujii@sony.com>
>+
>+        [WK2] fast/parser/document-open-in-unload.html makes the following test crash
>+        https://bugs.webkit.org/show_bug.cgi?id=98345
>+        <rdar://problem/12474923>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        m_documentLoader can become null in the middle of
>+        FrameLoader::commitProvisionalLoad by stopping the loading.
>+
>+        No new tests (Covered by existing tests).
>+
>+        * loader/FrameLoader.cpp:
>+        (WebCore::FrameLoader::commitProvisionalLoad): Do null check of m_documentLoader.
>+
> 2018-06-24  Tomas Popela  <tpopela@redhat.com>
> 
>         Unreviewed, address Darin's comment
>diff --git a/Source/WebCore/loader/FrameLoader.cpp b/Source/WebCore/loader/FrameLoader.cpp
>index 3f20375d716bd6e5ca4e4a64737295b489985a7f..e2f9ccd402bb0aafda200d92121d07ca2d85f0ec 100644
>--- a/Source/WebCore/loader/FrameLoader.cpp
>+++ b/Source/WebCore/loader/FrameLoader.cpp
>@@ -2004,7 +2004,9 @@ void FrameLoader::commitProvisionalLoad()
>     LOG(Loading, "WebCoreLoading %s: Finished committing provisional load to URL %s", m_frame.tree().uniqueName().string().utf8().data(),
>         m_frame.document() ? m_frame.document()->url().stringCenterEllipsizedToLength().utf8().data() : "");
> 
>-    if (m_loadType == FrameLoadType::Standard && m_documentLoader->isClientRedirect())
>+    if (m_loadType == FrameLoadType::Standard
>+        && m_documentLoader
>+        && m_documentLoader->isClientRedirect())
>         history().updateForClientRedirect();
> 
>     if (m_loadingFromCachedPage) {
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index 9d67f5c3f7366efc2084b74d4f421086966c220a..1a02fbb1b08c1eba20623a06f69e673cc9accd7e 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-06-25  Fujii Hironori  <Hironori.Fujii@sony.com>
>+
>+        [WK2] fast/parser/document-open-in-unload.html makes the following test crash
>+        https://bugs.webkit.org/show_bug.cgi?id=98345
>+        <rdar://problem/12474923>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * platform/wk2/TestExpectations: Unskipped fast/parser/document-open-in-unload.html.
>+
> 2018-06-25  Antoine Quint  <graouts@apple.com>
> 
>         [Web Animations] Make imported/mozilla/css-animations/test_animation-pausing.html pass reliably
>diff --git a/LayoutTests/platform/wk2/TestExpectations b/LayoutTests/platform/wk2/TestExpectations
>index 117a2fffd5b368363e19d724ea56ab1d52ed4771..8c3bc68312242f08cab9ba1cb7dc4a82f21bc3fd 100644
>--- a/LayoutTests/platform/wk2/TestExpectations
>+++ b/LayoutTests/platform/wk2/TestExpectations
>@@ -157,9 +157,6 @@ plugins/npruntime/embed-property-iframe-equality.html
> 
> webkit.org/b/105952 fast/loader/submit-form-while-parsing-2.html [ Pass Failure ]
> 
>-# Causes next test to crash only on WebKit2
>-webkit.org/b/98345 fast/parser/document-open-in-unload.html [ Skip ]
>-
> webkit.org/b/141122 editing/selection/programmatic-selection-on-mac-is-directionless.html [ Pass Failure ]
> 
> webkit.org/b/149087 http/tests/cache/disk-cache/disk-cache-cancel.html [ Pass Failure ]

Actions: View | Formatted Diff | Diff

Attachments on bug 98345: 343575 | 343577 | 343579 | 343584 | 343585 | 343701 | 343899 | 344067