[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdelibs/khtml/misc
From:       Simon Paul St James <kdedevel () etotheipiplusone ! com>
Date:       2008-08-16 17:32:39
Message-ID: 1218907959.372727.21748.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 848047 by sstjames:

Don't force all images to be re-downloaded if softReload is true.  Approved by \
ggarand.  The corresponding Konqueror patch that offers a "forced" reload action is \
awaiting review.  This probably isn't a full fix for 137720, but should be of \
interest to the reporter.

CCBUG:137720

 M  +4 -1      loader.cpp  


--- trunk/KDE/kdelibs/khtml/misc/loader.cpp #848046:848047
@@ -1079,7 +1079,10 @@
 bool DocLoader::needReload(CachedObject *existing, const QString& fullURL)
 {
     bool reload = false;
-    if (m_cachePolicy == KIO::CC_Verify)
+    if (m_cachePolicy == KIO::CC_Verify ||    
+        // During a softReload, we favour using cached images
+        // over forcibly re-downloading them.
+        (existing->type() == CachedObject::Image && \
m_part->browserExtension()->browserArguments().softReload))  {
        if (!m_reloadedURLs.contains(fullURL))
        {


[prev in list] [next in list] [prev in thread] [next in thread]