Title: [147267] trunk/Source/WebCore
Revision
147267
Author
commit-qu...@webkit.org
Date
2013-03-29 19:08:40 -0700 (Fri, 29 Mar 2013)

Log Message

Unreviewed, rolling out r147263.
http://trac.webkit.org/changeset/147263
https://bugs.webkit.org/show_bug.cgi?id=113632

Breaks test fast/loader/display-image-unset-allows-cached-
image-load.html (Requested by mlam on #webkit).

Patch by Sheriff Bot <webkit.review....@gmail.com> on 2013-03-29

* loader/cache/CachedResource.cpp:
(WebCore::CachedResource::freshnessLifetime):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (147266 => 147267)


--- trunk/Source/WebCore/ChangeLog	2013-03-30 01:49:00 UTC (rev 147266)
+++ trunk/Source/WebCore/ChangeLog	2013-03-30 02:08:40 UTC (rev 147267)
@@ -1,3 +1,15 @@
+2013-03-29  Sheriff Bot  <webkit.review....@gmail.com>
+
+        Unreviewed, rolling out r147263.
+        http://trac.webkit.org/changeset/147263
+        https://bugs.webkit.org/show_bug.cgi?id=113632
+
+        Breaks test fast/loader/display-image-unset-allows-cached-
+        image-load.html (Requested by mlam on #webkit).
+
+        * loader/cache/CachedResource.cpp:
+        (WebCore::CachedResource::freshnessLifetime):
+
 2013-03-29  Roger Fong  <roger_f...@apple.com>
 
         Unreviewed. AppleWin VS2010 build fix.

Modified: trunk/Source/WebCore/loader/cache/CachedResource.cpp (147266 => 147267)


--- trunk/Source/WebCore/loader/cache/CachedResource.cpp	2013-03-30 01:49:00 UTC (rev 147266)
+++ trunk/Source/WebCore/loader/cache/CachedResource.cpp	2013-03-30 02:08:40 UTC (rev 147267)
@@ -432,14 +432,7 @@
     
 double CachedResource::freshnessLifetime() const
 {
-    // Let file: resources expire immediately so that we don't serve a stale
-    // resource when a file has changed underneath us. Modern filesystems
-    // implement their own caches, so we should still get good performance if
-    // the resource hasn't changed.
-    if (m_response.url().protocolIs("file"))
-        return 0;
-
-    // Cache other non-http resources liberally.
+    // Cache non-http resources liberally
     if (!m_response.url().protocolIsInHTTPFamily())
         return std::numeric_limits<double>::max();
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to