617906 - Turn on IPC/OOPP in Thunderbird builds

Status: RESOLVED FIXED

Description

[Mark Banner (:standard8) (afk 15 - 18 Oct)]

We should turn on inter-process-communication and out-of-process-plugins for nightly builds. The main advantages at this stage are that it brings us closer to the Firefox core so that we're not diverging too far. This should help to reduce random bustages in our code.

The side-effect is that RSS feeds and content-tabs will be able to make use of OOPP if desired.

We'll also have the chance for separating out other processes later on, if I correctly understand the direction that IPC is going in

Comment 1

[Mark Banner (:standard8) (afk 15 - 18 Oct)]

Created attachment 517428 [details] [diff] [review]
[checked in] Part 1 - basic build config

Ok, this just ports to Thunderbird the various build config related parts to enable IPC.

I'm currently looking at doing the UI hooks. However, I think if m-c wants to land the restriction to stop disable IPC builds, then this would be enough (with a confvars tweak) to enable IPC.

Comment 2

[Justin Wood (:Callek)]

Comment on attachment 517428 [details] [diff] [review]
[checked in] Part 1 - basic build config

I wouldn't have expected those additions to package-manifest getting turned on by this bug, (except the obvious @MOZ_CHILD_PROCESS_NAME@) but I trust you to have added necessary files there, so rs+ for that part. (even if its just stuff you are having ride-along in this bug)

For a strictly, "do it, UI Later" this looks good.

Comment 3

[Mark Banner (:standard8) (afk 15 - 18 Oct)]

(In reply to comment #2)
> I wouldn't have expected those additions to package-manifest getting turned on
> by this bug, (except the obvious @MOZ_CHILD_PROCESS_NAME@) but I trust you to
> have added necessary files there, so rs+ for that part. (even if its just stuff
> you are having ride-along in this bug)

Ok, so they should probably have ifdef MOZ_IPC around them, which I guess I'll add but remove again in a few days.

Comment 4

[Mark Banner (:standard8) (afk 15 - 18 Oct)]

Comment on attachment 517428 [details] [diff] [review]
[checked in] Part 1 - basic build config

Checked in: http://hg.mozilla.org/comm-central/rev/519649224203

Comment 5

[Mark Banner (:standard8) (afk 15 - 18 Oct)]

Created attachment 521130 [details] [diff] [review]
[checked in] Part 2 - enable IPC

This works locally and I've currently got this running on try server. Although we've not got any of the UI strings for it (to say the plugin has crashed, or whatever), we can still run plugins with it.

Given that trunk is currently failing to build because of this, I think we can turn it on for both 3.3 and trunk and finish the UI later (shouldn't take long, but today isn't the day for me to try and get that working).

Comment 6

[David :Bienvenu]

Comment on attachment 521130 [details] [diff] [review]
[checked in] Part 2 - enable IPC

rs=me

Comment 7

[Mark Banner (:standard8) (afk 15 - 18 Oct)]

This is effectively fixed, I've raised bug 661410 to implement the UI for IPC/OOPP.