Bug 17470 – throws are not properly recorded in the console in onreadystatechange

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore JavaScript

[Francisco Tolmasky]

myXMLHTTPRequestObject.onreadystatechange = function () { throw "hello". }

This will report "undefined" in the error console, whereas it reports "hello" from everywhere else it could be thrown.

[Alexey Proskuryakov]

Created attachment 19256 [details]
test case

I'm interested in which cases work though - XMLHttpRequest, XPathEvaluator and a plain timeout all seem to work in the same way.

[Francisco Tolmasky]

(In reply to comment #1)
> Created an attachment (id=19256) [edit]
> test case
> 
> I'm interested in which cases work though - XMLHttpRequest, XPathEvaluator and
> a plain timeout all seem to work in the same way.
> 
Just doing
<script>
throw "yikes"
</script>

works

[Alexey Proskuryakov]

Right - apparently, all callbacks are affected.

This looks like an easy fix, but I'd like to look into providing line numbers and getting rid of code duplication at the same time.

[Alexey Proskuryakov]

See also: bug 18232, bug 18234.

[Francisco Tolmasky]

This basically makes debugging in Safari impossible.  I really think that this should be a P1 given that any bug you have in any ajax code requires you to do alert debugging essentially.

[David Kilzer (:ddkilzer)]

<rdar://problem/5908156>

[Francisco Tolmasky]

thank you kilzer

[Oliver Hunt]

Fixed in r35253