Bug 63506 - [JSC] window.onerror lacks file and line number information for ReferenceErrors
: [JSC] window.onerror lacks file and line number information for ReferenceErrors
Status: NEW
: WebKit
WebCore JavaScript
: 528+ (Nightly build)
: All All
: P2 Normal
Assigned To:
: http://jsErrLog.appspot.com
:
:
:
  Show dependency treegraph
 
Reported: 2011-06-27 20:23 PST by
Modified: 2012-08-03 13:01 PST (History)


Attachments
Attempt at a repro (339 bytes, text/html)
2011-06-27 22:00 PST, Dominic Cooney
no flags Details
Demonstrates the failure case described (446 bytes, text/html)
2011-06-27 22:25 PST, Offbeatmammal
no flags Details
screen shot of "foo" sample failing on OSX (108.74 KB, image/tiff)
2011-06-28 21:34 PST, Offbeatmammal
no flags Details


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2011-06-27 20:23:44 PST
with the fix to bug 8519 we now have a working window.onerror, and as implemented in current Chrome builds (also FF and IE) works consistently for all errors encountered so far.

In the current WebKit nightly (r81895) though this error fails to provide correct parameters

<script type="text/javascript">
    Messsage = "This is a test";    // note three s's
    document.write(Message);        // note two s's
</script>

... both the file_loc and the line_no that should be passed to the function are empty:

window.onerror=function(msg, file_loc, line_no){ }

sample available at http://blog.offbeatmammal.com/samples/jserr/jserr2.html (a working error, if that's not an oximoron, is at http://blog.offbeatmammal.com/samples/jserr/jserr.html)
------- Comment #1 From 2011-06-27 22:00:35 PST -------
Created an attachment (id=98857) [details]
Attempt at a repro
------- Comment #2 From 2011-06-27 22:02:14 PST -------
Works for me in WebKit nightly r89812.
------- Comment #3 From 2011-06-27 22:24:17 PST -------
(In reply to comment #2)
> Works for me in WebKit nightly r89812.

the sample repro works fine in r89815 (sorry about earlier typo) but if you change your reference to

foo();

to the same as the error above... it fails
(if I can work out how to add the sample I will)
------- Comment #4 From 2011-06-27 22:25:16 PST -------
Created an attachment (id=98859) [details]
Demonstrates the failure case described
------- Comment #5 From 2011-06-27 22:47:32 PST -------
Attachment 98859 [details] works as expected on Mac OS X in WebKit nightly r89812 on Mac but fails in WebKit nightly r89815 on Win.

This may be specific to JSC on Windows. There are no JSC changes in 89812-89815.
------- Comment #6 From 2011-06-28 21:34:34 PST -------
Created an attachment (id=99032) [details]
screen shot of "foo" sample failing on OSX

don't want to be a pain, but on OSX 10.6.7 with r89812 both the "foo" and the "messSage" samples fail to give either line number or script for me

Looks like the problem is both OSX and Windows
------- Comment #7 From 2011-08-15 14:17:44 PST -------
updated to remove [win] specific designation
------- Comment #8 From 2011-11-18 14:25:05 PST -------
It's also not working for me. I'm using openSUSE 11.4 (x86_64) and I just came across this bug.
------- Comment #9 From 2012-08-03 13:01:07 PST -------
appears to be working on Safari 6.0 on OSX 10.7.4 (my current test platform)
not working in Safari on iOS 5.1.1 (no access to 6.0 yet to test)