Bug 126109 - CStack: Introduce JSStack::ensureCapacityFor().
Summary: CStack: Introduce JSStack::ensureCapacityFor().
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mark Lam
URL:
Keywords:
Depends on:
Blocks: 125928
  Show dependency tree / graph
 
Reported: 2013-12-20 19:18 PST by Mark Lam
Modified: 2013-12-29 08:38 PST (History)
5 users (show)

See Also:

Attachments
the patch. (12.26 KB, patch)
2013-12-20 19:26 PST, Mark Lam 		msaboff: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Lam 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.
Comment 1 Mark Lam 2013-12-20 19:26:28 PST 
Created attachment 219832 [details]
the patch.
Comment 2 Mark Lam 2013-12-20 23:10:01 PST 
Landed in r160960 on the jsCStack: <http://trac.webkit.org/r160960>.

Still need a review.
Comment 3 Mark Lam 2013-12-29 08:38:27 PST 
Thanks for the review.  Review status updated in r161118: <http://trac.webkit.org/r161118>.