Bug 253388

Summary: REGRESSION (261182@main): TestWebKitAPI.ProcessSwap.LockdownModeEnabledByDefaultThenOptOut and TestWebKitAPI.ProcessSwap.NavigatingToLockdownMode are failing
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: New BugsAssignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, vitor.roriz, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Description Ryan Haddad 2023-03-04 09:32:26 PST
The following two tests are consistently failing after https://commits.webkit.org/261182@main

    TestWebKitAPI.ProcessSwap.LockdownModeEnabledByDefaultThenOptOut
        ERROR: Error while launching com.apple.WebKit.Networking: Connection invalid
        /Volumes/Data/worker/Apple-Ventura-Debug-Build/build/Source/WebKit/UIProcess/Launcher/cocoa/ProcessLauncherCocoa.mm(213) : auto WebKit::ProcessLauncher::launchProcess()::(anonymous class)::operator()(xpc_object_t) const
        LEAK: 2 WebProcessPool
        
        /Volumes/Data/worker/Apple-Ventura-Debug-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:7913
        Expected equality of these values:
          runJSCheck("!!window.CSSCounterStyleRule"_s)
            Which is: true
          shouldBeEnabled == ShouldBeEnabled::Yes
            Which is: false
        
        
        /Volumes/Data/worker/Apple-Ventura-Debug-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:7913
        Expected equality of these values:
          runJSCheck("!!window.CSSCounterStyleRule"_s)
            Which is: true
          shouldBeEnabled == ShouldBeEnabled::Yes
            Which is: false
        
        
        /Volumes/Data/worker/Apple-Ventura-Debug-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:7913
        Expected equality of these values:
          runJSCheck("!!window.CSSCounterStyleRule"_s)
            Which is: true
          shouldBeEnabled == ShouldBeEnabled::Yes
            Which is: false
        
    TestWebKitAPI.ProcessSwap.NavigatingToLockdownMode
        ERROR: Error while launching com.apple.WebKit.Networking: Connection invalid
        /Volumes/Data/worker/Apple-Ventura-Debug-Build/build/Source/WebKit/UIProcess/Launcher/cocoa/ProcessLauncherCocoa.mm(213) : auto WebKit::ProcessLauncher::launchProcess()::(anonymous class)::operator()(xpc_object_t) const
        LEAK: 1 WebProcessPool
        
        /Volumes/Data/worker/Apple-Ventura-Debug-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:7913
        Expected equality of these values:
          runJSCheck("!!window.CSSCounterStyleRule"_s)
            Which is: true
          shouldBeEnabled == ShouldBeEnabled::Yes
            Which is: false

https://results.webkit.org/?suite=api-tests&suite=api-tests&test=TestWebKitAPI.ProcessSwap.CannotDisableLockdownModeWithoutBrowserEntitlement&test=TestWebKitAPI.ProcessSwap.NavigatingToLockdownMode
Comment 1 Radar WebKit Bug Importer 2023-03-04 09:32:37 PST
<rdar://problem/106235827>
Comment 2 Brent Fulgham 2023-03-04 10:46:28 PST
We need to correct the expectations here now that the feature is expected to be on.
Comment 3 Brent Fulgham 2023-03-04 13:27:33 PST
Pull request: https://github.com/Webkit/WebKit/pull/11071
Comment 4 EWS 2023-03-04 14:42:54 PST
Committed 261231@main (93ddd026548d): <https://commits.webkit.org/261231@main>

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