Bug 161975

Summary: Use Options::validateExceptionChecks() instead of VM::m_verifyExceptionEvents.
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, keith_miller, msaboff, saam
Priority: P2    
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
proposed patch. keith_miller: review+

Mark Lam
Reported 2016-09-14 10:15:34 PDT
This makes it less burdensome (no longer needs a rebuild to enable checks) to do incremental work towards enabling checks all the time.
Attachments
proposed patch. (2.91 KB, patch)
2016-09-14 10:20 PDT, Mark Lam 		keith_miller: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2016-09-14 10:20:33 PDT
Created attachment 288827 [details] proposed patch.
Keith Miller
Comment 2 2016-09-14 10:47:38 PDT
Comment on attachment 288827 [details] proposed patch. r=me assuming no perf regressions if verifyExceptionCheckNeedIsSatisfied is even built in release.
Mark Lam
Comment 3 2016-09-14 10:58:21 PDT
Thanks for the review. Landed in r205917: <http://trac.webkit.org/r205917>.
Note You need to log in before you can comment on or make changes to this bug.