Patch to follow.
Created attachment 107157 [details] Patch
Comment on attachment 107157 [details] Patch lgtm
Comment on attachment 107157 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=107157&action=review > Source/WebCore/bindings/v8/ScriptDebugServer.cpp:254 > + v8::TryCatch tryCatch; What's the purpose of the trycatch here? The next call is not supposed to throw
> What's the purpose of the trycatch here? The next call is not supposed to throw I'm fixing the reproducible crash originating from the next call. Although it does not crash anymore due to the changes in the DebuggerScript.js, I'd like to put the try / catch here in order to cover potential changes to the javascript API that DebuggerScript is using.
(In reply to comment #4) > > What's the purpose of the trycatch here? The next call is not supposed to throw > > I'm fixing the reproducible crash originating from the next call. Although it does not crash anymore due to the changes in the DebuggerScript.js, I'd like to put the try / catch here in order to cover potential changes to the javascript API that DebuggerScript is using. Putting try/catch there will only make finding such problems, I suggest you revert this part.
Committed r95083: <http://trac.webkit.org/changeset/95083>