Bug 162797

Summary:

Remove the dumping of the stack back trace in VM::verifyExceptionCheckNeedIsSatisfied().

Product:

WebKit

Reporter:

Mark Lam <mark.lam>

Component:

JavaScriptCore

Assignee:

Mark Lam <mark.lam>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, keith_miller, msaboff, saam

Priority:

Version:

WebKit Local Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

Attachments:

Description	Flags
proposed patch.	ggaren: review+

Description Mark Lam 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.

Comment 1 Mark Lam 2016-09-30 11:36:00 PDT

Created attachment 290356 [details]
proposed patch.

Comment 2 Geoffrey Garen 2016-09-30 12:47:23 PDT

Comment on attachment 290356 [details]
proposed patch.

r=me

Comment 3 Mark Lam 2016-09-30 12:50:59 PDT

Thanks for the review.  Landed in r206658: <http://trac.webkit.org/r206658>.