WebKit2 should launch the WebProcess in 32-bit mode if the UIProcess is in 32-bit mode

[Sam Weinig]

WebKit2 should launch the WebProcess in 32-bit mode if the UIProcess is in 32-bit mode

[Sam Weinig]

<rdar://problem/8348100>

[Sam Weinig]

Created attachment 65362 [details]
Patch

[Mark Rowe (bdash)]

Comment on attachment 65362 [details]
Patch

> +
> +    posix_spawnattr_t attr;
> +    posix_spawnattr_init(&attr);
> +
> +#if CPU(X86)
> +    // We spawn a 32-bit child process if the host is 32-bit. This allows checking
> +    // the "Open in 32-bit mode" check box in the finder and getting a 32-bit WebProcess.

I don’t like the way this comment is worded.  It would be better to say something about having the child process run as the same architecture as the parent process.  The Finder in particular isn’t all that interesting here.

> +    cpu_type_t cpuTypes[] = { CPU_TYPE_X86 };    
> +#else
> +    cpu_type_t cpuTypes[] = { CPU_TYPE_ANY };
> +#endif
> +    size_t outCount = 0;
> +    posix_spawnattr_setbinpref_np(&attr, 1, cpuTypes, &outCount);

You could move this call in to the #if and drop the #else completely.

r=me

[Sam Weinig]

Landed in 65968.