Bug 143746

Summary: Remove JavaScriptCoreUseJIT default from JavaScriptCore
Product: WebKit Reporter: Michael Saboff <msaboff>
Component: JavaScriptCoreAssignee: Michael Saboff <msaboff>
Status: RESOLVED FIXED    
Severity: Normal CC: ggaren, mark.lam
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch mark.lam: review+

Michael Saboff
Reported 2015-04-14 17:49:32 PDT
This default is no longer useful. Safari used it at one point, but no longer can with webkit running as a separate process. rdar://problem/20470158
Attachments
Patch (1.39 KB, patch)
2015-04-14 17:53 PDT, Michael Saboff 		mark.lam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Michael Saboff
Comment 1 2015-04-14 17:53:14 PDT
Created attachment 250768 [details] Patch
Mark Lam
Comment 2 2015-04-14 17:59:05 PDT
Comment on attachment 250768 [details] Patch r=me. Just curious, what motivated the removal of this code? Is it simply because it's not needed anymore?
Mark Lam
Comment 3 2015-04-14 17:59:57 PDT
(In reply to comment #2) > Just curious, what motivated the removal of this code? Is it simply because > it's not needed anymore? Never mind. You already explained why in the bugzilla description.
Michael Saboff
Comment 4 2015-04-14 19:17:38 PDT
Committed r182831: <http://trac.webkit.org/changeset/182831>
Geoffrey Garen
Comment 5 2015-04-15 11:15:48 PDT
The motivation is memory use.
Note You need to log in before you can comment on or make changes to this bug.