596328 - non-IPC builds burning with nsContentPermissionHelper

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Bruno 'Aqualon' Escherl]

One of the landings from http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=d668a9be7bd4&tochange=df5f653ea413 makes our tree burn, probably because we don't build IPC.

Comment 1

[Josh Matthews [:jdm]]

Providing the errors would be helpful.

Comment 2

[Robert Kaiser]

Judging from the messages, which have us fail with nsContentPermissionHelper, this must be bug 595445 and/or bug 595456, and yes, I'm pretty sure that not having IPC enabled yet is the culprit.The bug belongs into DOM, where the failure is happening.

Comment 3

[Robert Kaiser]

Failure logs:

Linux:
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284490532.1284491031.5612.gz

In file included from /builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.cpp:38:
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:55: error: expected class-name before ‘{’ token
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:57: error: expected ‘,’ or ‘...’ before ‘::’ token
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:57: error: ISO C++ forbids declaration of ‘IPC’ with no type
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:60: error: ISO C++ forbids declaration of ‘nsCOMPtr’ with no type
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:60: error: expected ‘;’ before ‘<’ token
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:61: error: ISO C++ forbids declaration of ‘nsCOMPtr’ with no type
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:61: error: expected ‘;’ before ‘<’ token
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:62: error: ISO C++ forbids declaration of ‘nsCOMPtr’ with no type
/builds/slave/comm-central-trunk-linux/build/mozilla/dom/base/nsContentPermissionHelper.h:62: error: expected ‘;’ before ‘<’ token
NEXT ERROR make[6]: *** [nsContentPermissionHelper.o] Error 1
Mac:
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284488480.1284488776.25575.gz

In file included from /builds/slave/comm-central-trunk-linux64/build/mozilla/dom/base/nsContentPermissionHelper.cpp:38:0:
/builds/slave/comm-central-trunk-linux64/build/mozilla/dom/base/nsContentPermissionHelper.h:41:29: fatal error: base/basictypes.h: No such file or directory
compilation terminated.
make[6]: *** [nsContentPermissionHelper.o] Error 1

Windows:

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284490948.1284491711.9220.gz

e:\builds\slave\comm-central-trunk-win32\build\mozilla\dom\base\nsContentPermissionHelper.h(41) : fatal error C1083: Cannot open include file: 'base/basictypes.h': No such file or directory
make[6]: *** [nsContentPermissionHelper.obj] Error 2

Comment 4

[Bruno 'Aqualon' Escherl]

Thunderbird has the same problems, e.g. http://tinderbox.mozilla.org/showlog.cgi?log=Thunderbird/1284490126.1284491352.7238.gz

Comment 5

[Mark Banner (:standard8)]

I have a fix under test for this.

Comment 6

[Mark Banner (:standard8)]

Attachment: Bustage fix

Bustage fix for non-IPC builds - just move the ifdef to encompass everything.

Comment 7

[Benjamin Smedberg]

Comment on attachment 475187 [details] [diff] [review]
Bustage fix

I think that the correct fix here is to ifdef out this file in the Makefile.in, not here. This will cause build warnings about empty .o files, which may actually be a build error in some configurations.

Comment 8

[Mark Banner (:standard8)]

Attachment: Bustage fix v2

Knew I should have gone with the other version... This should fix it.

Comment 9

[Mark Banner (:standard8)]

I'm off to bed, so if this gets review and someone wants to land on my behalf, please do.

Comment 10

[Bruno 'Aqualon' Escherl]

(In reply to comment #8)
> Created attachment 475215 [details] [diff] [review]
> Bustage fix v2
> 
> Knew I should have gone with the other version... This should fix it.
Pushed by jdm as http://hg.mozilla.org/mozilla-central/rev/53a5148e8fad

00:08 < firebot> Thunderbird: 'WINNT 5.2 comm-central build' has changed state from Burning to Success.

-> Resolved Fixed