593328 - Only build hudservice for Firefox

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[Robert Kaiser]

As the Web Console is currently implemented in Firefox-specific ways, it's best if it's just not built for other applications.

I understand that going for bug 579909 fully and moving to browser/ is probably not in the cards in the rush to get a FF4 release stabilized, so just adding an ifdef to only build the service and tests for FF is probably the easiest thing. Having the theme files around in the JARs is not a problem, but e.g. having the tests run for other products or possibly add-ons wrongly believing they can hook into the Web Console is not what we should have in apps that can't support this.

I have an easy ifdef patch around, will attach in a minute.

Comment 1

[Robert Kaiser]

Attachment: just ifdef it FF-only

And here's the easy ifdef patch. In theory, I could test for MOZ_PHOENIX instead, but some time ago, I have heard that testing for MOZ_BUILD_APP is preferred. Still, I'll leave that up to review.

Comment 2

[Kevin Dangoor]

Reprioritizing bugs. You can filter the mail on the word TEABAGS.

Comment 3

[Shawn Wilsher :sdwilsh]

Comment on attachment 471816 [details] [diff] [review]
just ifdef it FF-only

I'm going to delegate this to ted since he'll be able to answer comment 1 better than I.

Comment 4

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 471816 [details] [diff] [review]
just ifdef it FF-only

Stealing.

Comment 5

[Mounir Lamouri (:mounir)]

Pushed:
http://hg.mozilla.org/mozilla-central/rev/2779c55431a4

Comment 6

[Wolfgang Rosenauer [:wolfiR]]

This broke "firefox on top of xulrunner" builds as toolkit is part of xulrunner but is not built as browser.

Comment 7

[Kevin Dangoor]

I opened bug 601907 as a followup to this to look into the Firefox on xulrunner problem.

Comment 8

[Wolfgang Rosenauer [:wolfiR]]

Oops, sorry, I discussed with khuey already and filed 601890 (forgot to mention here)