Enable XPC Service based WebProcess with runtime flag

[Sam Weinig]

Enable XPC Service base WebProcess with runtime flag

[Sam Weinig]

Created attachment 157885 [details]
Patch

Comment on attachment 157885 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=157885&action=review

> Source/WTF/ChangeLog:3
> +        Enable XPC Service base WebProcess with runtime flag

based?

> Source/WebKit2/ChangeLog:3
> +        Enable XPC Service base WebProcess with runtime flag

based?

> Tools/ChangeLog:3
> +        Enable XPC Service base WebProcess with runtime flag

based?

> Tools/Scripts/webkitdirs.pm:2697
> +  --guard-malloc                    Enable Guard Malloc (Mac OS X only)
> +  --use-web-process-xpc-service     Launch the WebProcess as an XPC Service (Mac OS X only)

Could drop “Mac”

> Tools/Scripts/webkitdirs.pm:2729
> +        $ENV{__XPC_DYLD_INSERT_LIBRARIES} = $productDir . "/WebProcessShim.dylib";

Shouldn’t this copy and append to DYLD_INSERT_LIBRARIES so that guard malloc works?
Shouldn’t the UI process unset these so that they don’t propagate to other processes it launches?

> Tools/Scripts/webkitdirs.pm:2772
> +            $ENV{__XPC_DYLD_FRAMEWORK_PATH} = $productDir;
> +            $ENV{__XPC_DYLD_INSERT_LIBRARIES} = $productDir . "/WebProcessShim.dylib";

Shouldn’t the UI process unset these so that they don’t propagate to other processes it launches?

> Tools/Scripts/webkitdirs.pm:2779
> +            die "Targetting the WebProcess is not compatible with using an XPC Service for the WebProcess at this time.";

I think it’s either “the Web process” or “WebProcess”, but not “the WebProcess”.

[Sam Weinig]

(In reply to comment #2)
> (From update of attachment 157885 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=157885&action=review
> 
> > Source/WTF/ChangeLog:3
> > +        Enable XPC Service base WebProcess with runtime flag
> 
> based?
> 
> > Source/WebKit2/ChangeLog:3
> > +        Enable XPC Service base WebProcess with runtime flag
> 
> based?
> 
> > Tools/ChangeLog:3
> > +        Enable XPC Service base WebProcess with runtime flag
> 
> based?
> 
> > Tools/Scripts/webkitdirs.pm:2697
> > +  --guard-malloc                    Enable Guard Malloc (Mac OS X only)
> > +  --use-web-process-xpc-service     Launch the WebProcess as an XPC Service (Mac OS X only)
> 
> Could drop “Mac”
> 
> > Tools/Scripts/webkitdirs.pm:2729
> > +        $ENV{__XPC_DYLD_INSERT_LIBRARIES} = $productDir . "/WebProcessShim.dylib";
> 
> Shouldn’t this copy and append to DYLD_INSERT_LIBRARIES so that guard malloc works?

Yes.

> Shouldn’t the UI process unset these so that they don’t propagate to other processes it launches?

Unfortunately, that does work with the __XPC variants of these. Before this is the default, we will have to find a way of doing it, but I don't have it yet. 

> 
> > Tools/Scripts/webkitdirs.pm:2772
> > +            $ENV{__XPC_DYLD_FRAMEWORK_PATH} = $productDir;
> > +            $ENV{__XPC_DYLD_INSERT_LIBRARIES} = $productDir . "/WebProcessShim.dylib";
> 
> Shouldn’t the UI process unset these so that they don’t propagate to other processes it launches?
> 
> > Tools/Scripts/webkitdirs.pm:2779
> > +            die "Targetting the WebProcess is not compatible with using an XPC Service for the WebProcess at this time.";
> 
> I think it’s either “the Web process” or “WebProcess”, but not “the WebProcess”.

Ok.

[Build Bot]

Comment on attachment 157885 [details]
Patch

Attachment 157885 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/13474814

[Sam Weinig]

Created attachment 157886 [details]
Patch

[Sam Weinig]

Created attachment 157887 [details]
Updated patch

[Sam Weinig]

Committed r125361: <http://trac.webkit.org/changeset/125361>