2009-03-29 Darin Adler <darin@apple.com>
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 30 Mar 2009 02:49:37 +0000 (02:49 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 30 Mar 2009 02:49:37 +0000 (02:49 +0000)
        Requested by Antti Koivisto.

        * loader/DocumentLoader.cpp:
        (WebCore::DocumentLoader::subresource): Tweaked code to be a little clearer and added
        a better comment.

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

WebCore/ChangeLog
WebCore/loader/DocumentLoader.cpp

index 46db76a..080f563 100644 (file)
@@ -1,5 +1,13 @@
 2009-03-29  Darin Adler  <darin@apple.com>
 
 2009-03-29  Darin Adler  <darin@apple.com>
 
+        Requested by Antti Koivisto.
+
+        * loader/DocumentLoader.cpp:
+        (WebCore::DocumentLoader::subresource): Tweaked code to be a little clearer and added
+        a better comment.
+
+2009-03-29  Darin Adler  <darin@apple.com>
+
         Reviewed by Dan Bernstein.
 
         Bug 23445: Copying certain hidden text causes a crash
         Reviewed by Dan Bernstein.
 
         Bug 23445: Copying certain hidden text causes a crash
index 119cc2a..9557526 100644 (file)
@@ -553,20 +553,12 @@ PassRefPtr<ArchiveResource> DocumentLoader::subresource(const KURL& url) const
     if (!resource || resource->preloadResult() == CachedResource::PreloadReferenced)
         return archiveResourceForURL(url);
 
     if (!resource || resource->preloadResult() == CachedResource::PreloadReferenced)
         return archiveResourceForURL(url);
 
-    bool wasPurgeable = resource->isPurgeable();
-    if (wasPurgeable) {
-        if (!resource->makePurgeable(false))
-            return 0;
-    }
+    // FIXME: This has the side effect of making the resource non-purgeable.
+    // It would be better if it didn't have this permanent affect.
+    if (!resource->makePurgeable(false))
+        return 0;
+
     RefPtr<SharedBuffer> data = resource->data();
     RefPtr<SharedBuffer> data = resource->data();
-    if (wasPurgeable) {
-        // At the time of this writing, the following operation is a no-op, because
-        // makePurgeable has no effect if someone is holding a reference to the
-        // SharedBuffer. So we could just leave it out, but it seems that some day
-        // we might want to change the design so this does have an effect, and
-        // this call is otherwise harmless.
-        resource->makePurgeable(true);
-    }
     if (!data)
         return 0;
 
     if (!data)
         return 0;