Bug 263592 - REGRESSION (269698@main): 3 TestWebKitAPI.ProcessSwap LockdownMode API tests are failing
Summary: REGRESSION (269698@main): 3 TestWebKitAPI.ProcessSwap LockdownMode API tests ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-10-24 08:40 PDT by Ryan Haddad
Modified: 2023-10-24 13:26 PDT (History)
9 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2023-10-24 08:40:35 PDT 
Then following two tests are failing on iOS and macOS after https://commits.webkit.org/269698@main landed

TestWebKitAPI.ProcessSwap.NavigatingToLockdownMode
TestWebKitAPI.ProcessSwap.CannotDisableLockdownModeWithoutBrowserEntitlement 


    TestWebKitAPI.ProcessSwap.LockdownModeEnabledByDefaultThenOptOut
        /Volumes/Data/worker/Apple-Monterey-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:8001Expected equality of these values:  runJSCheck("!!window.Highlight"_s)    Which is: true  shouldBeEnabled == ShouldBeEnabled::Yes    Which is: false/Volumes/Data/worker/Apple-Monterey-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:8001Expected equality of these values:  runJSCheck("!!window.Highlight"_s)    Which is: true  shouldBeEnabled == ShouldBeEnabled::Yes    Which is: false/Volumes/Data/worker/Apple-Monterey-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:8001Expected equality of these values:  runJSCheck("!!window.Highlight"_s)    Which is: true  shouldBeEnabled == ShouldBeEnabled::Yes    Which is: false
    TestWebKitAPI.ProcessSwap.NavigatingToLockdownMode
        /Volumes/Data/worker/Apple-Monterey-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:8001Expected equality of these values:  runJSCheck("!!window.Highlight"_s)    Which is: true  shouldBeEnabled == ShouldBeEnabled::Yes    Which is: false

https://results.webkit.org/?suite=api-tests&suite=api-tests&test=TestWebKitAPI.ServiceWorker.WindowClientNavigateCrossOrigin&test=TestWebKitAPI.ProcessSwap.NavigatingToLockdownMode
Comment 1 Radar WebKit Bug Importer 2023-10-24 08:40:50 PDT 
<rdar://problem/117417946>
Comment 2 Ryan Haddad 2023-10-24 08:52:57 PDT Comment hidden (obsolete)
Comment 3 Ryan Haddad 2023-10-24 09:43:17 PDT 
These are the tests that are failing across iOS and macOS 

TestWebKitAPI.ProcessSwap.NavigatingToLockdownMode
TestWebKitAPI.ProcessSwap.CannotDisableLockdownModeWithoutBrowserEntitlement
TestWebKitAPI.ProcessSwap.LockdownModeEnabledByDefaultThenOptOut

https://results.webkit.org/?suite=api-tests&suite=api-tests&suite=api-tests&test=TestWebKitAPI.ProcessSwap.CannotDisableLockdownModeWithoutBrowserEntitlement&test=TestWebKitAPI.ProcessSwap.LockdownModeEnabledByDefaultThenOptOut&test=TestWebKitAPI.ProcessSwap.NavigatingToLockdownMode
Comment 4 EWS 2023-10-24 13:26:15 PDT 
Committed 269729@main (c3397b4f9ae9): <https://commits.webkit.org/269729@main>

Reviewed commits have been landed. Closing PR #19496 and removing active labels.