[webkit-changes] [WebKit/WebKit] 52b154: TestWebKitAPI.WKWebExtensionAPITabs.ActiveTab is f...
Timothy Hatcher
noreply at github.com
Mon Dec 18 14:07:44 PST 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 52b154a911f1b2450d8d73569b70599a6b6e9974
https://github.com/WebKit/WebKit/commit/52b154a911f1b2450d8d73569b70599a6b6e9974
Author: Timothy Hatcher <timothy at apple.com>
Date: 2023-12-18 (Mon, 18 Dec 2023)
Changed paths:
M Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionAPITabs.mm
M Tools/TestWebKitAPI/cocoa/WebExtensionUtilities.mm
Log Message:
-----------
TestWebKitAPI.WKWebExtensionAPITabs.ActiveTab is flaky.
https://webkit.org/b/266594
rdar://119346255
Reviewed by Brian Weinstein.
The URL does not change for tabs during the provisional load, and saying it was changing
was causing the onChanged event to fire too soon for this test to be accurate.
Also tweak how we check for properties in the changeInfo object to be consistent.
* Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionAPITabs.mm:
(TestWebKitAPI::TEST):
* Tools/TestWebKitAPI/cocoa/WebExtensionUtilities.mm:
(-[TestWebExtensionTab webView:didStartProvisionalNavigation:]):
(-[TestWebExtensionTab webView:didReceiveServerRedirectForProvisionalNavigation:]):
(-[TestWebExtensionTab webView:didFailNavigation:withError:]):
Canonical link: https://commits.webkit.org/272240@main
More information about the webkit-changes
mailing list