Bug 143746 - Remove JavaScriptCoreUseJIT default from JavaScriptCore
Summary: Remove JavaScriptCoreUseJIT default from JavaScriptCore
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Michael Saboff
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-04-14 17:49 PDT by Michael Saboff
Modified: 2015-04-15 11:15 PDT (History)
2 users (show)

See Also:

Attachments
Patch (1.39 KB, patch)
2015-04-14 17:53 PDT, Michael Saboff 		mark.lam: 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 Michael Saboff 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
Comment 1 Michael Saboff 2015-04-14 17:53:14 PDT 
Created attachment 250768 [details]
Patch
Comment 2 Mark Lam 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?
Comment 3 Mark Lam 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.
Comment 4 Michael Saboff 2015-04-14 19:17:38 PDT 
Committed r182831: <http://trac.webkit.org/changeset/182831>
Comment 5 Geoffrey Garen 2015-04-15 11:15:48 PDT 
The motivation is memory use.