143746 2015-04-14 17:49:32 -0700 Remove JavaScriptCoreUseJIT default from JavaScriptCore 2015-04-15 11:15:48 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 msaboff msaboff ggaren mark.lam oldest_to_newest 1085364 0 msaboff 2015-04-14 17:49:32 -0700 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 1085366 1 250768 msaboff 2015-04-14 17:53:14 -0700 Created attachment 250768 Patch 1085367 2 250768 mark.lam 2015-04-14 17:59:05 -0700 Comment on attachment 250768 Patch r=me. Just curious, what motivated the removal of this code? Is it simply because it's not needed anymore? 1085368 3 mark.lam 2015-04-14 17:59:57 -0700 (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. 1085387 4 msaboff 2015-04-14 19:17:38 -0700 Committed r182831: <http://trac.webkit.org/changeset/182831> 1085601 5 ggaren 2015-04-15 11:15:48 -0700 The motivation is memory use. 250768 2015-04-14 17:53:14 -0700 2015-04-14 17:59:05 -0700 Patch 143746.patch text/plain 1424 msaboff SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTgyODI3KQorKysgU291cmNl L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEzIEBA CisyMDE1LTA0LTE0ICBNaWNoYWVsIFNhYm9mZiAgPG1zYWJvZmZAYXBwbGUuY29tPgorCisgICAg ICAgIFJlbW92ZSBKYXZhU2NyaXB0Q29yZVVzZUpJVCBkZWZhdWx0IGZyb20gSmF2YVNjcmlwdENv cmUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0Mzc0 NgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogcnVu dGltZS9WTS5jcHA6CisgICAgICAgIChKU0M6OmVuYWJsZUFzc2VtYmxlcik6CisKIDIwMTUtMDQt MTQgIE1pY2hhZWwgU2Fib2ZmICA8bXNhYm9mZkBhcHBsZS5jb20+CiAKICAgICAgICAgREZHIHJl Z2lzdGVyIGZpbGxTcGVjdWxhdGUqKCkgZnVuY3Rpb25zIHNob3VsZCB2YWxpZGF0ZSBpbmNvbWlu ZyBzcGlsbCBmb3JtYXQgaXMgY29tcGF0aWJsZSB3aXRoIHJlcXVlc3RlZCBmaWxsIGZvcm1hdApJ bmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvVk0uY3BwCj09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9ydW50aW1lL1ZNLmNwcAkocmV2aXNpb24gMTgyODI2KQor KysgU291cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvVk0uY3BwCSh3b3JraW5nIGNvcHkpCkBA IC0xMjcsMTMgKzEyNyw2IEBAIHN0YXRpYyBib29sIGVuYWJsZUFzc2VtYmxlcihFeGVjdXRhYmxl QWwKICAgICAgICAgcmV0dXJuIGZhbHNlOwogICAgIH0KIAotI2lmIFVTRShDRikKLSAgICBDRlN0 cmluZ1JlZiBjYW5Vc2VKSVRLZXkgPSBDRlNUUigiSmF2YVNjcmlwdENvcmVVc2VKSVQiKTsKLSAg ICBSZXRhaW5QdHI8Q0ZUeXBlUmVmPiBjYW5Vc2VKSVQgPSBhZG9wdENGKENGUHJlZmVyZW5jZXND b3B5QXBwVmFsdWUoY2FuVXNlSklUS2V5LCBrQ0ZQcmVmZXJlbmNlc0N1cnJlbnRBcHBsaWNhdGlv bikpOwotICAgIGlmIChjYW5Vc2VKSVQpCi0gICAgICAgIHJldHVybiBrQ0ZCb29sZWFuVHJ1ZSA9 PSBjYW5Vc2VKSVQuZ2V0KCk7Ci0jZW5kaWYKLQogI2lmIFVTRShDRikgfHwgT1MoVU5JWCkKICAg ICBjaGFyKiBjYW5Vc2VKSVRTdHJpbmcgPSBnZXRlbnYoIkphdmFTY3JpcHRDb3JlVXNlSklUIik7 CiAgICAgcmV0dXJuICFjYW5Vc2VKSVRTdHJpbmcgfHwgYXRvaShjYW5Vc2VKSVRTdHJpbmcpOwo=