REGRESSION: http/tests/cache/disk-cache/disk-cache-revalidation-new-expire-header...
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Aug 2015 20:53:26 +0000 (20:53 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Aug 2015 20:53:26 +0000 (20:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=148205

Unreviewed, give the disk cache a chance to settle down before querying
the resource again. This fixes the flakiness locally. Longer term, I will
try and figure out why the cache is sometimes revalidating if the resource
is requested very quickly after.

* http/tests/cache/disk-cache/disk-cache-revalidation-new-expire-header.html:

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

LayoutTests/ChangeLog
LayoutTests/http/tests/cache/disk-cache/disk-cache-revalidation-new-expire-header.html

index 5d973d0..904a527 100644 (file)
@@ -1,3 +1,15 @@
+2015-08-20  Chris Dumez  <cdumez@apple.com>
+
+        REGRESSION: http/tests/cache/disk-cache/disk-cache-revalidation-new-expire-header.html is very flaky
+        https://bugs.webkit.org/show_bug.cgi?id=148205
+
+        Unreviewed, give the disk cache a chance to settle down before querying
+        the resource again. This fixes the flakiness locally. Longer term, I will
+        try and figure out why the cache is sometimes revalidating if the resource
+        is requested very quickly after.
+
+        * http/tests/cache/disk-cache/disk-cache-revalidation-new-expire-header.html:
+
 2015-08-11  Yusuke Suzuki  <utatane.tea@gmail.com>
 
         Introduce put_by_id like IC into put_by_val when the given name is String or Symbol
index 033ca0f..6b42041 100644 (file)
@@ -13,8 +13,11 @@ description("Test that the 'Expires' header is updated upon successful validatio
 debug("");
 
 runTests(tests, function() {
-    debug("304 response included an 'Expires' header in the future, so we should not need to revalidate this time.");
-    runTests(tests);
+    // Wait for things to settle down in the cache.
+    setTimeout(function() {
+        debug("304 response included an 'Expires' header in the future, so we should not need to revalidate this time.");
+        runTests(tests);
+    }, 200);
 });
 
 </script>