17470 2008-02-21 03:16:31 -0800 throws are not properly recorded in the console in onreadystatechange 2008-07-20 05:47:39 -0700 1 1 1 Unclassified WebKit WebCore JavaScript 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED InRadar P2 Normal --- 1 tolmasky ap ap jchaffraix oldest_to_newest 71375 0 tolmasky 2008-02-21 03:16:31 -0800 myXMLHTTPRequestObject.onreadystatechange = function () { throw "hello". } This will report "undefined" in the error console, whereas it reports "hello" from everywhere else it could be thrown. 71384 1 19256 ap 2008-02-21 06:09:59 -0800 Created attachment 19256 test case I'm interested in which cases work though - XMLHttpRequest, XPathEvaluator and a plain timeout all seem to work in the same way. 71417 2 tolmasky 2008-02-21 13:02:49 -0800 (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 71419 3 ap 2008-02-21 13:28:49 -0800 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. 76031 4 ap 2008-03-31 22:58:12 -0700 See also: bug 18232, bug 18234. 78240 5 tolmasky 2008-04-19 01:04:50 -0700 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. 79496 6 ddkilzer 2008-05-02 15:27:46 -0700 <rdar://problem/5908156> 79502 7 tolmasky 2008-05-02 15:51:35 -0700 thank you kilzer 86459 8 oliver 2008-07-20 05:47:39 -0700 Fixed in r35253 19256 2008-02-21 06:09:59 -0800 2008-02-21 06:09:59 -0800 test case onreadystatechange-throw.html text/html 559 ap PHNjcmlwdD4KICAgIHZhciByZXEgPSBuZXcgWE1MSHR0cFJlcXVlc3Q7CiAgICByZXEub25yZWFk eXN0YXRlY2hhbmdlID0gZnVuY3Rpb24gKCkgeyB0aHJvdyAib25yZWFkeXN0YXRlY2hhbmdlIjsg fQogICAgcmVxLm9wZW4oIkdFVCIsICJvbnJlYWR5c3RhdGVjaGFuZ2UtdGhyb3cuaHRtbCIpOwog IAogICAgc2V0VGltZW91dChmdW5jdGlvbigpIHsgdGhyb3cgInRpbWVvdXQiOyB9LCAwKTsKCiAg ICBmdW5jdGlvbiBNeU5TUmVzb2x2ZXIoKSB7fQogICAgTXlOU1Jlc29sdmVyLnByb3RvdHlwZS5s b29rdXBOYW1lc3BhY2VVUkkgPSBmdW5jdGlvbihwcmVmaXgpIHsKICAgICAgICB0aHJvdyAiTlNS ZXNvbHZlciI7CiAgICB9CgogICAgdHJ5IHsKICAgICAgICBkb2N1bWVudC5ldmFsdWF0ZSgiZm9v OmNoaWxkL3RleHQoKSIsIGRvY3VtZW50LmRvY3VtZW50RWxlbWVudCwgbmV3IE15TlNSZXNvbHZl ciwgCiAgICAgICAgICAgIFhQYXRoUmVzdWx0Lk9SREVSRURfTk9ERV9JVEVSQVRPUl9UWVBFLCBu dWxsKTsKICAgIH0gY2F0Y2ggKGV4KQogICAgewogICAgfQoKPC9zY3JpcHQ+Cg==