Bug 170312

Summary: WebAssembly: Strip WasmParser and WasmFunctionParser from knowing about VM
Product: WebKit Reporter: Saam Barati <saam>
Component: JavaScriptCoreAssignee: Saam Barati <saam>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, fpizlo, ggaren, gskachkov, jfbastien, keith_miller, mark.lam, msaboff, ticaiolima, ysuzuki
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 168264    
Attachments:
Description Flags
patch mark.lam: review+

Saam Barati
Reported 2017-03-30 16:27:04 PDT
...
Attachments
patch (11.79 KB, patch)
2017-03-31 15:45 PDT, Saam Barati 		mark.lam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Saam Barati
Comment 1 2017-03-30 17:11:31 PDT
This will mean Signature should no longer care about VM. Should be easy.
Saam Barati
Comment 2 2017-03-31 15:45:49 PDT
Created attachment 306021 [details] patch
JF Bastien
Comment 3 2017-03-31 15:50:45 PDT
Comment on attachment 306021 [details] patch lol why was there ever a VM? lgtm
Mark Lam
Comment 4 2017-03-31 16:05:06 PDT
Comment on attachment 306021 [details] patch r=me
Saam Barati
Comment 5 2017-03-31 16:08:21 PDT
landed in: https://trac.webkit.org/changeset/214695/webkit
Note You need to log in before you can comment on or make changes to this bug.