Commit a9b9735b authored by beidson@apple.com's avatar beidson@apple.com

2009-06-23 Brady Eidson <beidson@apple.com>

        Patch by Antti Koivisto.
        Reviewed by Brady Eidson.

        <rdar://problem/6988010> Failed resources not reloaded when built against Tiger SDK

        A latent bug that cache revalidation uncovered.  When a CachedResource failed to load,
        it should always be revalidated upon reload.

        * loader/CachedResource.cpp:
        (WebCore::CachedResource::canUseCacheValidator):
        (WebCore::CachedResource::mustRevalidate):



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@45031 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 1062b2ef
2009-06-23 Brady Eidson <beidson@apple.com>
Patch by Antti Koivisto.
Reviewed by Brady Eidson.
<rdar://problem/6988010> Failed resources not reloaded when built against Tiger SDK
A latent bug that cache revalidation uncovered. When a CachedResource failed to load,
it should always be revalidated upon reload.
* loader/CachedResource.cpp:
(WebCore::CachedResource::canUseCacheValidator):
(WebCore::CachedResource::mustRevalidate):
2009-06-23 Dan Bernstein <mitz@apple.com>
Reviewed by Darin Adler.
......
......@@ -373,7 +373,7 @@ void CachedResource::updateResponseAfterRevalidation(const ResourceResponse& val
bool CachedResource::canUseCacheValidator() const
{
if (m_loading)
if (m_loading || m_errorOccurred)
return false;
if (m_response.cacheControlContainsNoStore())
......@@ -386,6 +386,9 @@ bool CachedResource::canUseCacheValidator() const
bool CachedResource::mustRevalidate(CachePolicy cachePolicy) const
{
if (m_errorOccurred)
return true;
if (m_loading)
return false;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment