[webkit-reviews] review granted: [Bug 62614] rwt: handle worker exceptions cleanly : [Attachment 97161] update w/ review feedback from tony

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 14 13:30:03 PDT 2011


Tony Chang <tony at chromium.org> has granted Dirk Pranke <dpranke at chromium.org>'s
request for review:
Bug 62614: rwt: handle worker exceptions cleanly
https://bugs.webkit.org/show_bug.cgi?id=62614

Attachment 97161: update w/ review feedback from tony
https://bugs.webkit.org/attachment.cgi?id=97161&action=review

------- Additional Comments from Tony Chang <tony at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=97161&action=review

> Tools/Scripts/webkitpy/layout_tests/layout_package/manager.py:702
>	       raise

Yes, you're right, this gives the right behavior.  I was confused.  If you were
to raise e, the caught exception, it would lose the stack.


More information about the webkit-reviews mailing list