Bug 234613

Summary: Re-enable CustomDisplayName and DefaultDisplayName API tests on Monterey
Product: WebKit Reporter: Alex Christensen <achristensen>
Component: New BugsAssignee: Alex Christensen <achristensen>
Status: RESOLVED FIXED    
Severity: Normal CC: beidson, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=234858
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Alex Christensen
Reported 2021-12-22 11:29:48 PST
Re-enable CustomDisplayName and DefaultDisplayName API tests on Monterey
Attachments
Patch (13.92 KB, patch)
2021-12-22 11:31 PST, Alex Christensen
no flags
Patch (12.60 KB, patch)
2021-12-22 13:57 PST, Alex Christensen
no flags
Patch (13.32 KB, patch)
2021-12-22 15:00 PST, Alex Christensen
no flags
Patch (12.93 KB, patch)
2022-01-24 13:44 PST, Alex Christensen
no flags
Alex Christensen
Comment 1 2021-12-22 11:31:00 PST
Alex Christensen
Comment 2 2021-12-22 11:34:06 PST
Alex Christensen
Comment 3 2021-12-22 13:57:19 PST
Alex Christensen
Comment 4 2021-12-22 15:00:28 PST
Alex Christensen
Comment 5 2021-12-22 15:13:44 PST
Comment on attachment 447836 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=447836&action=review > Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm:521 > + bool sendToNetworkProcess = isInProcessInitialization == IsInProcessInitialization::No needs semicolon
Alex Christensen
Comment 6 2021-12-22 15:16:22 PST
Ryan Haddad
Comment 7 2022-01-04 16:19:34 PST
Reverted r287374 for reason: Caused assertion failures on Monterey Committed r287595 (245723@trunk): <https://commits.webkit.org/245723@trunk>
Radar WebKit Bug Importer
Comment 8 2022-01-04 16:20:23 PST
Ryan Haddad
Comment 9 2022-01-04 16:23:21 PST
Alex Christensen
Comment 10 2022-01-19 22:16:08 PST
It looks like this test can only be run on pre-Monterey macOS or internal macOS with the necessary entitlement. Otherwise we would need to start the network process to keep the right state, but we can't start the network process yet without breaking other things.
Alex Christensen
Comment 11 2022-01-24 13:44:02 PST
EWS
Comment 12 2022-01-24 22:10:43 PST
Committed r288540 (246372@main): <https://commits.webkit.org/246372@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 449850 [details].
Note You need to log in before you can comment on or make changes to this bug.