REGRESSION(r141136): Apple's internal PLT test suite doesn't finish
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 Feb 2013 21:11:35 +0000 (21:11 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 Feb 2013 21:11:35 +0000 (21:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=108380

Reviewed by Alexey Proskuryakov.

Source/WebCore:

Re-enable the main resource cache since the regression had been fixed in r141615.

* loader/cache/CachedResourceLoader.cpp:
(WebCore::CachedResourceLoader::requestResource):

LayoutTests:

Re-enable tests that have been temporarily disabled.

* platform/mac/TestExpectations:

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

LayoutTests/ChangeLog
LayoutTests/platform/mac/TestExpectations
Source/WebCore/ChangeLog
Source/WebCore/loader/cache/CachedResourceLoader.cpp

index 2cedf7b39f80934be4a03dffb24ac354b92d0d1b..f86b949b757e2e86e6d09b53775db8402244bc9d 100644 (file)
@@ -1,3 +1,14 @@
+2013-02-06  Ryosuke Niwa  <rniwa@webkit.org>
+
+        REGRESSION(r141136): Apple's internal PLT test suite doesn't finish
+        https://bugs.webkit.org/show_bug.cgi?id=108380
+
+        Reviewed by Alexey Proskuryakov.
+
+        Re-enable tests that have been temporarily disabled.
+
+        * platform/mac/TestExpectations:
+
 2013-02-06  Uday Kiran  <udaykiran@motorola.com>
 
         Implement 'vmax' from CSS3 values and units
index fcfb5f5036c4354402681c0a32ad7812e6c7ce3f..3e3d4c41278dc0020f1cedb51134db854998a7d9 100644 (file)
@@ -1329,10 +1329,3 @@ webkit.org/b/108370 editing/spelling/spelling-with-whitespace-selection.html [ S
 
 webkit.org/b/108257 [ Debug ] compositing/overflow/composited-scrolling-creates-a-stacking-container.html [ Crash ]
 webkit.org/b/108257 [ Debug ] compositing/overflow/automatically-opt-into-composited-scrolling.html [ Crash ]
-
-webkit.org/b/108380 http/tests/cache/cached-main-resource.html [ Failure ]
-webkit.org/b/108380 http/tests/cache/iframe-304-crash.html [ Failure ]
-webkit.org/b/108380 http/tests/inspector/resource-har-pages.html [ Failure ]
-webkit.org/b/108380 http/tests/loading/redirect-methods.html [ Failure ]
-webkit.org/b/108380 http/tests/misc/link-rel-icon-beforeload.html [ Failure ]
-webkit.org/b/108380 http/tests/misc/favicon-loads-with-images-disabled.html [ Failure ]
index 052e955f4604154fa7ec5f26633d2d0452687481..8c5a5f61b065ced9855a84b88c7f8a9f10ded3d8 100644 (file)
@@ -1,3 +1,15 @@
+2013-02-06  Ryosuke Niwa  <rniwa@webkit.org>
+
+        REGRESSION(r141136): Apple's internal PLT test suite doesn't finish
+        https://bugs.webkit.org/show_bug.cgi?id=108380
+
+        Reviewed by Alexey Proskuryakov.
+
+        Re-enable the main resource cache since the regression had been fixed in r141615.
+
+        * loader/cache/CachedResourceLoader.cpp:
+        (WebCore::CachedResourceLoader::requestResource):
+
 2013-02-06  Uday Kiran  <udaykiran@motorola.com>
 
         Implement 'vmax' from CSS3 values and units
index 2e4f6575d0291e1e9a8789aac8b7d4d4b4be2768..74323f9fc2161d832bd3ef67a35f558ee15996ab 100644 (file)
@@ -479,10 +479,9 @@ CachedResourceHandle<CachedResource> CachedResourceLoader::requestResource(Cache
         }
     }
 
-#if PLATFORM(CHROMIUM) || PLATFORM(MAC)
+#if PLATFORM(CHROMIUM)
     // FIXME: Temporarily leave main resource caching disabled for chromium, see https://bugs.webkit.org/show_bug.cgi?id=107962
     // Ensure main resources aren't preloaded, and other main resource loads are removed from cache to prevent reuse.
-    // FIXME: Temporarily leave main resource caching disabled on Mac port per webkit.org/b/108380 until we track down the root cause.
     if (type == CachedResource::MainResource) {
         ASSERT(policy != Use);
         ASSERT(policy != Revalidate);