2009-03-04 Brady Eidson <beidson@apple.com>
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 4 Mar 2009 18:25:50 +0000 (18:25 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 4 Mar 2009 18:25:50 +0000 (18:25 +0000)
commit44642f755ee8e254190ef63656c55057053a004d
tree3f79274ce65c0c05513197ecebb8861af49ebfca
parent2a1069dfd0bcb438bcfd130ae1fd3f3dae897f24
2009-03-04  Brady Eidson  <beidson@apple.com>

        Reviewed by Darin Adler

        <rdar://problem/6619630> Quick Look of vCards stuck on image of first card opened.

        I narrowed this down to http://trac.webkit.org/changeset/39304 which, among other things,
        consolidated some of the various decision making pieces of the Cache into the new method
        FrameLoader::cachePolicy().

        Before 39304, when deciding whether to use an existing CachedResource, we checked if the FrameLoader
        is reloading.  If it is, we'd evict any existing resource then recreate it.  Quick looks uses the
        same URL for this image every time and expects it to be reloaded with each new card.

        The FrameLoader::isReloading() check did one thing - Ask the DocumentLoader if it's cache policy
        is "ReloadIgnoringCacheData".  This check was lost in the consolidation to the new method.

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::cachePolicy): Restore the DocumentLoader's cachePolicy check as a possible
          condition for returning CachePolicyReload.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41425 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog
WebCore/loader/FrameLoader.cpp