Title: [150210] trunk/Source/WebCore
Revision
150210
Author
[email protected]
Date
2013-05-16 14:32:31 -0700 (Thu, 16 May 2013)

Log Message

Do not indefinitely cache resources from blob URLs.

Reviewed by Alexey Proskuryakov.

Alexey pointed out after I landed r150169 that blob URLs backed by
files should return an error if the file changed after the URL was
created. By indefinitely caching them in memory, we don't give the
loader a chance to check for modification. Remove "blob" from the list
of schemes that should be indefinitely cached.

* platform/SchemeRegistry.cpp:
(WebCore::SchemeRegistry::shouldCacheResponsesFromURLSchemeIndefinitely):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (150209 => 150210)


--- trunk/Source/WebCore/ChangeLog	2013-05-16 21:23:44 UTC (rev 150209)
+++ trunk/Source/WebCore/ChangeLog	2013-05-16 21:32:31 UTC (rev 150210)
@@ -1,3 +1,18 @@
+2013-05-16  Andy Estes  <[email protected]>
+
+        Do not indefinitely cache resources from blob URLs.
+
+        Reviewed by Alexey Proskuryakov.
+
+        Alexey pointed out after I landed r150169 that blob URLs backed by
+        files should return an error if the file changed after the URL was
+        created. By indefinitely caching them in memory, we don't give the
+        loader a chance to check for modification. Remove "blob" from the list
+        of schemes that should be indefinitely cached.
+
+        * platform/SchemeRegistry.cpp:
+        (WebCore::SchemeRegistry::shouldCacheResponsesFromURLSchemeIndefinitely):
+
 2013-05-16  Claudio Saavedra  <[email protected]>
 
         [CSS] CSS Variables are case-sensitive

Modified: trunk/Source/WebCore/platform/SchemeRegistry.cpp (150209 => 150210)


--- trunk/Source/WebCore/platform/SchemeRegistry.cpp	2013-05-16 21:23:44 UTC (rev 150209)
+++ trunk/Source/WebCore/platform/SchemeRegistry.cpp	2013-05-16 21:32:31 UTC (rev 150210)
@@ -326,7 +326,7 @@
     if (equalIgnoringCase(scheme, "applewebdata"))
         return true;
 #endif
-    return equalIgnoringCase(scheme, "blob") || equalIgnoringCase(scheme, "data");
+    return equalIgnoringCase(scheme, "data");
 }
 
 } // namespace WebCore
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to