Title: [143681] trunk/Source/WebCore
Revision
143681
Author
e...@webkit.org
Date
2013-02-21 18:54:25 -0800 (Thu, 21 Feb 2013)

Log Message

LayoutTests/fast/encoding/parser-tests-*.html timeout with threaded HTML parser
https://bugs.webkit.org/show_bug.cgi?id=109995

Unreviewed speculative fix for Mac WK1.

It appears that m_frame can be null for Mac WK1.

* loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::isLoading):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (143680 => 143681)


--- trunk/Source/WebCore/ChangeLog	2013-02-22 02:48:49 UTC (rev 143680)
+++ trunk/Source/WebCore/ChangeLog	2013-02-22 02:54:25 UTC (rev 143681)
@@ -1,3 +1,15 @@
+2013-02-21  Eric Seidel  <e...@webkit.org>
+
+        LayoutTests/fast/encoding/parser-tests-*.html timeout with threaded HTML parser
+        https://bugs.webkit.org/show_bug.cgi?id=109995
+
+        Unreviewed speculative fix for Mac WK1.
+
+        It appears that m_frame can be null for Mac WK1.
+
+        * loader/DocumentLoader.cpp:
+        (WebCore::DocumentLoader::isLoading):
+
 2013-02-21  Dean Jackson  <d...@apple.com>
 
         Better sizing model for Snapshotted plugins

Modified: trunk/Source/WebCore/loader/DocumentLoader.cpp (143680 => 143681)


--- trunk/Source/WebCore/loader/DocumentLoader.cpp	2013-02-22 02:48:49 UTC (rev 143680)
+++ trunk/Source/WebCore/loader/DocumentLoader.cpp	2013-02-22 02:54:25 UTC (rev 143681)
@@ -280,7 +280,7 @@
 
 bool DocumentLoader::isLoading() const
 {
-    if (m_frame->document() && m_frame->document()->hasActiveParser())
+    if (m_frame && m_frame->document() && m_frame->document()->hasActiveParser())
         return true;
     return isLoadingMainResource() || !m_subresourceLoaders.isEmpty() || !m_plugInStreamLoaders.isEmpty();
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to