Bug 233028

Summary: Refactor allocateCell() and tryAllocateCell() to take VM& instead of Heap&.
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: alecflett, beidson, cdumez, ews-watchlist, hi, joepeck, jsbell, keith_miller, msaboff, pangle, saam, tzagallo, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 232849    
Attachments:
Description Flags
proposed patch.
ews-feeder: commit-queue-
proposed patch.
ews-feeder: commit-queue-
proposed patch. ysuzuki: review+, ews-feeder: commit-queue-

Description Mark Lam 2021-11-11 21:34:14 PST 
allocateCell() and tryAllocateCell() can fetch Heap& from VM& themselves.  This allows us to make all the client code more terse, and also makes it easier to redirect the underlying heap to allocate from later for global GC work.  Additionally, this will help make the global GC patch less of a mega patch and easier to review later.
Comment 1 Radar WebKit Bug Importer 2021-11-11 21:34:57 PST 
<rdar://problem/85329680>
Comment 2 Mark Lam 2021-11-11 21:42:32 PST 
Created attachment 444042 [details]
proposed patch.
Comment 3 Mark Lam 2021-11-11 22:06:00 PST 
Created attachment 444045 [details]
proposed patch.
Comment 4 Mark Lam 2021-11-11 22:46:06 PST 
Created attachment 444049 [details]
proposed patch.
Comment 5 Yusuke Suzuki 2021-11-12 01:01:16 PST 
Comment on attachment 444049 [details]
proposed patch.

r=me
Comment 6 Mark Lam 2021-11-12 10:12:09 PST 
The mac-wk1 EWS failed tests is due to a known flaky crash on that test fast/dom/Geolocation/cached-position-iframe.html.  Hence, it's not related to this patch.
Comment 7 Mark Lam 2021-11-12 10:17:39 PST 
Thanks for the review.  Landed in r285730: <http://trac.webkit.org/r285730>.