Summary: | Introduce ExceptionScope::assertNoException() and releaseAssertNoException(). | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Lam <mark.lam> | ||||
Component: | JavaScriptCore | Assignee: | Mark Lam <mark.lam> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | buildbot, fpizlo, jfbastien, jlewis3, keith_miller, msaboff, ryanhaddad, saam, ysuzuki | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Mark Lam
2017-05-06 15:14:34 PDT
Created attachment 309300 [details]
proposed patch.
Let's try this on the EWS.
Comment on attachment 309300 [details] proposed patch. Attachment 309300 [details] did not pass jsc-ews (mac): Output: http://webkit-queues.webkit.org/results/3689922 New failing tests: ChakraCore.yaml/ChakraCore/test/Strings/HTMLHelpers.js.default (In reply to Build Bot from comment #2) > Comment on attachment 309300 [details] > proposed patch. > > Attachment 309300 [details] did not pass jsc-ews (mac): > Output: http://webkit-queues.webkit.org/results/3689922 > > New failing tests: > ChakraCore.yaml/ChakraCore/test/Strings/HTMLHelpers.js.default I can't reproduce this locally. Will rebase and try again. (In reply to Mark Lam from comment #3) > (In reply to Build Bot from comment #2) > > Comment on attachment 309300 [details] > > proposed patch. > > > > Attachment 309300 [details] did not pass jsc-ews (mac): > > Output: http://webkit-queues.webkit.org/results/3689922 > > > > New failing tests: > > ChakraCore.yaml/ChakraCore/test/Strings/HTMLHelpers.js.default > > I can't reproduce this locally. Will rebase and try again. Nope, not bogus. I can reproduce it now after I rebased to ToT. Probably not due to my patch, but I'll investigate. (In reply to Mark Lam from comment #4) > (In reply to Mark Lam from comment #3) > > (In reply to Build Bot from comment #2) > > > Comment on attachment 309300 [details] > > > proposed patch. > > > > > > Attachment 309300 [details] did not pass jsc-ews (mac): > > > Output: http://webkit-queues.webkit.org/results/3689922 > > > > > > New failing tests: > > > ChakraCore.yaml/ChakraCore/test/Strings/HTMLHelpers.js.default > > > > I can't reproduce this locally. Will rebase and try again. > > Nope, not bogus. I can reproduce it now after I rebased to ToT. Probably > not due to my patch, but I'll investigate. OK, I found out that: 1. the issue reproduces without my patch. 2. the issue is intermittent and doesn't always reproduce. Given that, I don't think it should block this patch. Let's get a review. (In reply to Mark Lam from comment #5) > > Nope, not bogus. I can reproduce it now after I rebased to ToT. Probably > > not due to my patch, but I'll investigate. > > OK, I found out that: > 1. the issue reproduces without my patch. > 2. the issue is intermittent and doesn't always reproduce. FYI, I filed https://bugs.webkit.org/show_bug.cgi?id=171786 to track the ChakraCore test failure bug. Comment on attachment 309300 [details]
proposed patch.
r=me.
Thanks for the review. Landed in r216428: <http://trac.webkit.org/r216428>. (In reply to Mark Lam from comment #8) > Thanks for the review. Landed in r216428: <http://trac.webkit.org/r216428>. It appears that this change broke the Windows Debug build: https://build.webkit.org/builders/Apple%20Win%20Debug%20%28Build%29/builds/1120 (In reply to Ryan Haddad from comment #9) > (In reply to Mark Lam from comment #8) > > Thanks for the review. Landed in r216428: <http://trac.webkit.org/r216428>. > > It appears that this change broke the Windows Debug build: > https://build.webkit.org/builders/Apple%20Win%20Debug%20%28Build%29/builds/ > 1120 Speculative build fix landed in r216444: <http://trac.webkit.org/r216444>. |