Add WKNavigationDelegate SPI exposing WebProcess crash reason
Created attachment 330747 [details] Patch
Comment on attachment 330747 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=330747&action=review > Source/WebKit/UIProcess/Cocoa/NavigationState.mm:894 > +static _WKProcessTerminationReason toWKProcessTerminationReason(ProcessTerminationReason reason) Nit - I think WebKit style guidelines recommend not having the "to" prefix (https://webkit.org/code-style-guidelines/#names-verb). Maybe processTerminationReason is sufficient? > Source/WebKit/UIProcess/Cocoa/NavigationState.mm:927 > + if (m_navigationState.m_navigationDelegateMethods.webViewWebContentProcessDidTerminateWithReason) { Nit - Maybe this should go ahead of the webViewWebContentProcessDidTerminate check, so that if a delegate implements both versions, the SPI version is preferred. Or maybe this doesn't really matter :P
Created attachment 330779 [details] Patch
http://trac.webkit.org/r226610
<rdar://problem/36367341>