Bug 162797

Summary: Remove the dumping of the stack back trace in VM::verifyExceptionCheckNeedIsSatisfied().
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   
Bug Depends on:    
Bug Blocks: 162351    
Attachments:
Description Flags
proposed patch. ggaren: review+

Mark Lam
Reported 2016-09-30 11:32:30 PDT
This is because the RELEASE_ASSERT() that follows immediately after will also dump the stack back trace. Hence, the first dump will be redundant.
Attachments
proposed patch. (1.84 KB, patch)
2016-09-30 11:36 PDT, Mark Lam 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2016-09-30 11:36:00 PDT
Created attachment 290356 [details] proposed patch.
Geoffrey Garen
Comment 2 2016-09-30 12:47:23 PDT
Comment on attachment 290356 [details] proposed patch. r=me
Mark Lam
Comment 3 2016-09-30 12:50:59 PDT
Thanks for the review. Landed in r206658: <http://trac.webkit.org/r206658>.
Note You need to log in before you can comment on or make changes to this bug.