Unreviewed, rolling out r143695.
authorzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Feb 2013 13:34:42 +0000 (13:34 +0000)
committerzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Feb 2013 13:34:42 +0000 (13:34 +0000)
http://trac.webkit.org/changeset/143695
https://bugs.webkit.org/show_bug.cgi?id=110554

Crashes in DocumentLoader::checkLoadComplete on AppleMac WK1, EFL, GTK.

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

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

Source/WebCore/ChangeLog
Source/WebCore/loader/DocumentLoader.cpp

index 03c8093..927f0fb 100644 (file)
@@ -1,3 +1,14 @@
+2013-02-22  Zan Dobersek  <zdobersek@igalia.com>
+
+        Unreviewed, rolling out r143695.
+        http://trac.webkit.org/changeset/143695
+        https://bugs.webkit.org/show_bug.cgi?id=110554
+
+        Crashes in DocumentLoader::checkLoadComplete on AppleMac WK1, EFL, GTK.
+
+        * loader/DocumentLoader.cpp:
+        (WebCore::DocumentLoader::isLoading):
+
 2013-02-22  Balazs Kelemen  <kbalazs@webkit.org>
 
         [CoordGfx] Minor cleanup in CoordinatedGraphicsScene::paintToGraphicsContext
index c4c66f8..377323e 100644 (file)
@@ -280,13 +280,8 @@ void DocumentLoader::commitIfReady()
 
 bool DocumentLoader::isLoading() const
 {
-    // FIXME: This should always be enabled, but it seems to cause
-    // http/tests/security/feed-urls-from-remote.html to timeout on Mac WK1
-    // see http://webkit.org/b/110554 and http://webkit.org/b/110401
-#if ENABLE(THREADED_HTML_PARSER)
     if (m_frame && m_frame->document() && m_frame->document()->hasActiveParser())
         return true;
-#endif
     return isLoadingMainResource() || !m_subresourceLoaders.isEmpty() || !m_plugInStreamLoaders.isEmpty();
 }