WebKitTestRunner bundleID should not be cleared before every test
authorkatherine_cheney@apple.com <katherine_cheney@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 9 Mar 2020 23:59:40 +0000 (23:59 +0000)
committerkatherine_cheney@apple.com <katherine_cheney@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 9 Mar 2020 23:59:40 +0000 (23:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=208823
<rdar://problem/60204539>

Reviewed by Per Arne Vollan.

This is causing test failures for layout tests which rely on the Info.plist
bundle identifier (not the one set via the override method).

* WebKitTestRunner/TestController.cpp:
(WTR::TestController::createWebViewWithOptions):
(WTR::TestController::ensureViewSupportsOptionsForTest):
* WebKitTestRunner/cocoa/TestControllerCocoa.mm:

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

Tools/ChangeLog
Tools/WebKitTestRunner/TestController.cpp

index bda4f3a..3abd792 100644 (file)
@@ -1,3 +1,19 @@
+2020-03-09  Kate Cheney  <katherine_cheney@apple.com>
+
+        WebKitTestRunner bundleID should not be cleared before every test
+        https://bugs.webkit.org/show_bug.cgi?id=208823
+        <rdar://problem/60204539>
+
+        Reviewed by Per Arne Vollan.
+
+        This is causing test failures for layout tests which rely on the Info.plist 
+        bundle identifier (not the one set via the override method).
+
+        * WebKitTestRunner/TestController.cpp:
+        (WTR::TestController::createWebViewWithOptions):
+        (WTR::TestController::ensureViewSupportsOptionsForTest):
+        * WebKitTestRunner/cocoa/TestControllerCocoa.mm:
+
 2020-03-09  Per Arne Vollan  <pvollan@apple.com>
 
         [macOS] Notification observer is not removed
index 4dac2e6..71c358a 100644 (file)
@@ -640,6 +640,7 @@ void TestController::createWebViewWithOptions(const TestOptions& options)
         exit(1);
     }
     if (!options.applicationBundleIdentifier.isEmpty()) {
+        clearApplicationBundleIdentifierTestingOverride();
         setApplicationBundleIdentifier(options.applicationBundleIdentifier);
         m_hasSetApplicationBundleIdentifier = true;
     }
@@ -812,9 +813,6 @@ void TestController::ensureViewSupportsOptionsForTest(const TestInvocation& test
         m_createdOtherPage = false;
     }
 
-#if PLATFORM(COCOA)
-    clearApplicationBundleIdentifierTestingOverride();
-#endif
 
     createWebViewWithOptions(options);