Fix content extension test flakiness.
authorachristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 20 Apr 2015 18:48:41 +0000 (18:48 +0000)
committerachristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 20 Apr 2015 18:48:41 +0000 (18:48 +0000)
https://bugs.webkit.org/show_bug.cgi?id=143950

Reviewed by Brady Eidson.

Tools:

* WebKitTestRunner/mac/TestControllerMac.mm:
(WTR::TestController::platformResetPreferencesToConsistentValues):
Remove the TestContentExtensions if it exists.
(WTR::TestController::platformConfigureViewForTest):
Log errors in case there are any.

LayoutTests:

* platform/mac-wk2/TestExpectations:
Mark contentextensions tests as not flaky any more.

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

LayoutTests/ChangeLog
LayoutTests/platform/mac-wk2/TestExpectations
Tools/ChangeLog
Tools/WebKitTestRunner/mac/TestControllerMac.mm

index 7246f76..865d82e 100644 (file)
@@ -1,3 +1,13 @@
+2015-04-20  Alex Christensen  <achristensen@webkit.org>
+
+        Fix content extension test flakiness.
+        https://bugs.webkit.org/show_bug.cgi?id=143950
+
+        Reviewed by Brady Eidson.
+
+        * platform/mac-wk2/TestExpectations:
+        Mark contentextensions tests as not flaky any more.
+
 2015-04-20  Beth Dakin  <bdakin@apple.com>
 
         Should not list these tests twice.
index a55cbfc..95c609f 100644 (file)
@@ -526,5 +526,4 @@ webkit.org/b/141085 [ Yosemite ] http/tests/media/video-query-url.html [ Pass Ti
 ### END OF (7) New WebKit2-only failures in Yosemite
 ########################################
 
-# Once flakiness is fixed, the expectation should be changed to [ Pass ], because these tests are disabled, and need to be re-enabled.
-webkit.org/b/143950 http/tests/contentextensions [ Pass Failure ]
+http/tests/contentextensions [ Pass ]
index eca6f0e..7033dcb 100644 (file)
@@ -1,5 +1,18 @@
 2015-04-20  Alex Christensen  <achristensen@webkit.org>
 
+        Fix content extension test flakiness.
+        https://bugs.webkit.org/show_bug.cgi?id=143950
+
+        Reviewed by Brady Eidson.
+
+        * WebKitTestRunner/mac/TestControllerMac.mm:
+        (WTR::TestController::platformResetPreferencesToConsistentValues):
+        Remove the TestContentExtensions if it exists.
+        (WTR::TestController::platformConfigureViewForTest):
+        Log errors in case there are any.
+
+2015-04-20  Alex Christensen  <achristensen@webkit.org>
+
         Properly report errors from _WKUserContentExtensionStore.
         https://bugs.webkit.org/show_bug.cgi?id=143808
 
index 0e42c4a..d2581a1 100644 (file)
@@ -85,6 +85,14 @@ static bool shouldUseThreadedScrolling(const TestInvocation& test)
 void TestController::platformResetPreferencesToConsistentValues()
 {
 #if WK_API_ENABLED
+    __block bool doneRemoving = false;
+    [[_WKUserContentExtensionStore defaultStore] removeContentExtensionForIdentifier:@"TestContentExtensions" completionHandler:^(NSError *error)
+    {
+        if (error)
+            NSLog(@"%@", [error helpAnchor]);
+        doneRemoving = true;
+    }];
+    platformRunUntil(doneRemoving, 0);
     [[_WKUserContentExtensionStore defaultStore] _removeAllContentExtensions];
 #endif
 }
@@ -114,14 +122,16 @@ void TestController::platformConfigureViewForTest(const TestInvocation& test)
         return;
     
 #if WK_API_ENABLED
-    __block bool done = false;
+    __block bool doneCompiling = false;
     [[_WKUserContentExtensionStore defaultStore] compileContentExtensionForIdentifier:@"TestContentExtensions" encodedContentExtension:contentExtensionString completionHandler:^(_WKUserContentFilter *filter, NSError *error)
     {
         if (!error)
             WKPageGroupAddUserContentFilter(WKPageGetPageGroup(TestController::singleton().mainWebView()->page()), (__bridge WKUserContentFilterRef)filter);
-        done = true;
+        else
+            NSLog(@"%@", [error helpAnchor]);
+        doneCompiling = true;
     }];
-    platformRunUntil(done, 0);
+    platformRunUntil(doneCompiling, 0);
 #endif
 }