WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
161975
Use Options::validateExceptionChecks() instead of VM::m_verifyExceptionEvents.
https://bugs.webkit.org/show_bug.cgi?id=161975
Summary
Use Options::validateExceptionChecks() instead of VM::m_verifyExceptionEvents.
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+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug