349220 - Move SeaMonkey browser chrome to suite/

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect)

Description

[Robert Kaiser]

This is the next episode of the SeaMonkey chrome consolidation in the suite/ directory: Now it's time to get browser chrome over there.

Patch following soon...

Comment 1

[Robert Kaiser]

BTW, I'm also planning to remove navExtraOverlay while I'm at it, as this one is completely empty (presumably it's been there for Netscape only).

Comment 2

[Robert Kaiser]

Attachment: cvs copies for moving browser files to suite/

This is the cvs copy file for this move, I have roughly tested it by feeding it to mcp and it did the right thing (and found all files to copy).

Comment 3

[Robert Kaiser]

Attachment: jar.mn patch for moving

This is the jar.mn patch for building the files from suite/ instead of xpfe/ - this removes xpfe/browser/jar.mn (which is suite-only) and also patches navigator.xul to not use the omitted overlay (be sure to have copied it before so the patch applies).
I still need to test if it works correctly with both xpfe-based suite and suiterunner.

Comment 4

[Robert Kaiser]

Comment on attachment 234489 [details] [diff] [review]
jar.mn patch for moving

This does not work yet like I'd want it to. Apart from a small error in a jar.mn, we also need some Makefile changes.

Comment 5

[Robert Kaiser]

Attachment: build browser chrome from suite/

OK, here's a patch that is tested to work and really makes us build the browser UI from suite/ for both xpfe suite and suiterunner.

Comment 6

[[e-mail]]

Comment on attachment 234513 [details] [diff] [review]
build browser chrome from suite/

>+*  content/navigator-platform/contents.rdf                          (contents-platform.rdf)
Note that for some reason the current contents-platform.rdf files contain platform-specific strings, so you'll need to neutralise the wording as per the version in suite/common. r+sr=me with this fixed.

Comment 7

[Robert Kaiser]

Checked in, and cvs removed all files from xpfe/browser/resources as that content has all moved now to suite/