Changeset 143681 in webkit

Timestamp:

Feb 21, 2013, 6:54:25 PM (12 years ago)

Author:

eric@webkit.org

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

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/DocumentLoader.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-02-21  Eric Seidel  <eric@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  <dino@apple.com>
 

trunk/Source/WebCore/loader/DocumentLoader.cpp

@@ -281 +281 @@
 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();