Don't make a sandbox extension for the bundle path if there is no bundle
Created attachment 283922 [details] Patch
Created attachment 283923 [details] Patch
Comment on attachment 283923 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=283923&action=review > Source/WebKit2/UIProcess/Cocoa/WebProcessPoolCocoa.mm:262 > + return [[[NSBundle mainBundle] bundlePath] stringByStandardizingPath]; I think you meant to use the mainBundle local variable here rather than calling [NSBundle mainBundle] again. > Source/WebKit2/UIProcess/Cocoa/WebProcessPoolCocoa.mm:263 > + return emptyString(); I suggest doing early exit instead of nesting the success case inside an if statement.
Comment on attachment 283923 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=283923&action=review > Source/WebKit2/UIProcess/Cocoa/WebProcessPoolCocoa.mm:261 > + if ([mainBundle bundleIdentifier]) This did not fix the original problem. I don't know if I'll land it. This was returning true where I though it would not.