Attention needed from Mark Mentovai
Justin Cohen added 1 comment![Open in Gerrit]()
Patchset-level comments
File-level comment, Patchset 9 (Latest):Justin Cohen .
resolvedThis is need for using Blink on iOS. I'm not sure why PROTECTED is required by Apple for Blink on iOS, or even if the in-process design for iOS makes sense for Blink on iOS.
If it does, this appears to work, but my big open questions are:
- (1) when to use EXCEPTION_STATE_IDENTITY_PROTECTED vs EXCEPTION_STATE_IDENTITY [perhaps as a flag passed to Crashpad, or maybe it's fine to use on anyone > iOS18]
- (2) Why the PROTECTED API doesn't pass the task and thread like the non-PROTECTED version, and if it's OK to simply recover it to match the old API, like I do in this CL (util/mach/exc_server_variants.cc::469-513)
- (3) Perhaps more minor, but the difference between ExcTraits and MachExcTraits each having their own kMachMessageIDExceptionRaiseStateIdentityProtected const (with the ExcTraits version unused)
Consider this is a first draft in what perhaps should be a conversation. Let me know if doing this over CL comments is OK, or if you'd prefer to chat.
Thanks!
Open in GerritRelated details
Attention is currently required from:
Gerrit-MessageType: comment
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I228784ed765a6b996473ed07e2c0f34f14f93d37
Gerrit-Change-Number: 7204428
Gerrit-PatchSet: 9
Gerrit-Comment-Date: Tue, 02 Dec 2025 17:47:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No