Bug 207410

Summary: [bmalloc] VMHeap can be merge into Heap
Product: WebKit Reporter: Basuke Suzuki <Basuke.Suzuki>
Component: bmallocAssignee: Basuke Suzuki <Basuke.Suzuki>
Status: RESOLVED FIXED    
Severity: Normal CC: annulen, Basuke.Suzuki, commit-queue, ews-watchlist, ggaren, gyuyoung.kim, ryuan.choi, sergio, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
PATCH none

Description Basuke Suzuki 2020-02-07 14:45:54 PST 
I don't know about the history of VMHeap, but it has only one member function in it and Heap is the only client of that function. No member variable is defined. It does nothing special with its class. It is safe to merge the function into Heap.
Comment 1 Basuke Suzuki 2020-02-07 15:14:07 PST 
Created attachment 390132 [details]
PATCH
Comment 2 Yusuke Suzuki 2020-02-08 01:52:29 PST 
Comment on attachment 390132 [details]
PATCH

r=me, I think VMHeap is no longer necessary too.
Comment 3 WebKit Commit Bot 2020-02-08 02:35:44 PST 
Comment on attachment 390132 [details]
PATCH

Clearing flags on attachment: 390132

Committed r256088: <https://trac.webkit.org/changeset/256088>
Comment 4 WebKit Commit Bot 2020-02-08 02:35:46 PST 
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2020-02-08 02:36:14 PST 
<rdar://problem/59285145>