[webkit-reviews] review granted: [Bug 131520] Add user default for FTL JIT : [Attachment 229095] Fix

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Apr 10 18:09:49 PDT 2014


Benjamin Poulain <benjamin at webkit.org> has granted Gavin Barraclough
<barraclough at apple.com>'s request for review:
Bug 131520: Add user default for FTL JIT
https://bugs.webkit.org/show_bug.cgi?id=131520

Attachment 229095: Fix
https://bugs.webkit.org/attachment.cgi?id=229095&action=review

------- Additional Comments from Benjamin Poulain <benjamin at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=229095&action=review


Could you remove the environment variable thingy?
Did you perf test this??

> Source/WebKit2/Shared/WebProcessCreationParameters.h:141
> +    

Extra whitespace here!

> Source/WebKit2/UIProcess/mac/WebContextMac.mm:101
> +    

ditto.

> Source/WebKit2/UIProcess/mac/WebContextMac.mm:171
> +    

ditto.


More information about the webkit-reviews mailing list