Created attachment 39032 [details] the patch Why does Completion.cpp include "unistd.h"? It's not used at all.
Comment on attachment 39032 [details] the patch r=me with one small fix below. > diff --git a/JavaScriptCore/ChangeLog b/JavaScriptCore/ChangeLog > + https://bugs.webkit.org/show_bug.cgi?id=(OOPS!) Please fill in the bug number when landing. It appears that this include was added for http://bugs.webkit.org/show_bug.cgi?id=17339 due to the use of getpid() in this file but that function call is no longer present.
landed @48048