| Summary: | [V8] Exception from JavaScript propagates to main script | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Adam Barth <abarth> | ||||
| Component: | WebCore JavaScript | Assignee: | Adam Barth <abarth> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | abarth, ap, dglazkov, japhet | ||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| URL: | http://webblaze.org/abarth/tests/jswrite/test3.html | ||||||
| Attachments: |
|
||||||
|
Description
Adam Barth
2009-06-15 22:29:27 PDT
I cannot reproduce this with Safari 3.2.3 or with a local debug build of r44713. This bug appears to be specific to the V8 bindings. I'll look. Thanks for the report. Nate, can you check on this? Any progress on this Nate? Should be an easy fix. In the lost time, Nate made some progress getting to the root cause of this bug. He's going to touch base with the V8 folks and confirm his understanding. Created attachment 38536 [details]
Patch to isolate compile time exceptions in javascript URLs
The problem is that we only isolate runtime exceptions and not compile time exceptions in javascript URLs.
Comment on attachment 38536 [details]
Patch to isolate compile time exceptions in javascript URLs
This looks great. The only problem is you indented using two spaces instead of four. :)
Someone can either fix this when landing manually, or you can post an updated patch that we can land automatically.
I'll land this manually. Committed r47753: <http://trac.webkit.org/changeset/47753> |