The error object (the fifth parameter for the window.onerror event handler) should ALWAYS return an @ deliminator. Because WebKit/Safari doesn't in order to parse and clean up the error details for proper error reporting developers have to add pointless additional logic. Gecko browsers always use the @ deliminator; if no function is used the deliminator[0] index is simply empty though the deliminator[1] index is the same for all stack trace indexes. This would make the error object compatible with existing browser engines and likely the only need is to remove whatever excessive logic exists in the code that conditionally outputs the @ deliminator so this should be easy for someone familiar with the engine to fix. The general overview of what is happening plus a prerequisite function for my platform: function array_remove_empty(a) { var k = []; for (var i = 0; i < a.length; i++) {if (a[i] == '') {k.push(i);}} var r = array_remove_keys(a,k); return r; } window.onerror = function(msg,url,line,col,error) { a = array_trim(array_remove_empty(a.split('\n').reverse())); console.log(a); } The text string output: 0 "http://1.0.0.1/scripts/index.js?1612087732:7859:96" 1 "window_onload@http://1.0.0.1/scripts/index.js?1612087732:7368:5" 2 "az19@http://1.0.0.1/scripts/index.js?1612087732:7364:9" As demonstrated [0] index lacks the @ deliminator creating pointless complications. Tested with the latest version of Safari 14 as of 2021-01-31.
While Error.stack format is not standardized (this means any format is OK), maybe, having @ for source URL case is not so bad idea.
Created attachment 418830 [details] Patch
Comment on attachment 418830 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=418830&action=review > Source/JavaScriptCore/ChangeLog:8 > + Append '@' if URL exists even if function name does not exist. Can you say why you want this here?
Comment on attachment 418830 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=418830&action=review >> Source/JavaScriptCore/ChangeLog:8 >> + Append '@' if URL exists even if function name does not exist. > > Can you say why you want this here? Added :)
Created attachment 418831 [details] Patch
Created attachment 418843 [details] Patch
Committed r272139: <https://trac.webkit.org/changeset/272139> All reviewed patches have been landed. Closing bug and clearing flags on attachment 418843 [details].
<rdar://problem/73823423>