Skip to content
Permalink
Browse files
REGRESSION(r143664, r143681): http/tests/security/feed-urls-from-remo…
…te.html fails

https://bugs.webkit.org/show_bug.cgi?id=110554

Reviewed by Ryosuke Niwa.

Will fix http/tests/security/feed-urls-from-remote.html for Mac WK1.

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


Canonical link: https://commits.webkit.org/128837@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@143695 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
eseidel committed Feb 22, 2013
1 parent da61c1d commit c2bbab359ee08d039daf8ff5d43b9077ef5f1b0e
Showing with 17 additions and 0 deletions.
  1. +12 −0 Source/WebCore/ChangeLog
  2. +5 −0 Source/WebCore/loader/DocumentLoader.cpp
@@ -1,3 +1,15 @@
2013-02-21 Eric Seidel <eric@webkit.org>

REGRESSION(r143664, r143681): http/tests/security/feed-urls-from-remote.html fails
https://bugs.webkit.org/show_bug.cgi?id=110554

Reviewed by Ryosuke Niwa.

Will fix http/tests/security/feed-urls-from-remote.html for Mac WK1.

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

2013-02-21 Alec Flett <alecflett@chromium.org>

IndexedDB: Implement SharedBuffer version of put() / onSuccess()
@@ -280,8 +280,13 @@ 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();
}

0 comments on commit c2bbab3

Please sign in to comment.