Bug 126109

Summary: CStack: Introduce JSStack::ensureCapacityFor().
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: fpizlo, ggaren, mhahnenberg, msaboff, oliver
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 125928    
Attachments:
Description Flags
the patch. msaboff: review+

Mark Lam
Reported 2013-12-20 19:18:49 PST
JSStack::ensureCapacityFor() will be able to handle both the JS stack on the C stack or a separate stack. Clients should use JSStack::ensureCapacity() when doing stack checks for JS stack needs in place of JSStack::grow(). JSStack::grow() is now private.
Attachments
the patch. (12.26 KB, patch)
2013-12-20 19:26 PST, Mark Lam 		msaboff: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2013-12-20 19:26:28 PST
Created attachment 219832 [details] the patch.
Mark Lam
Comment 2 2013-12-20 23:10:01 PST
Landed in r160960 on the jsCStack: <http://trac.webkit.org/r160960>. Still need a review.
Mark Lam
Comment 3 2013-12-29 08:38:27 PST
Thanks for the review. Review status updated in r161118: <http://trac.webkit.org/r161118>.
Note You need to log in before you can comment on or make changes to this bug.