353391 - Camino ships 2 copies of the table images

Status: RESOLVED FIXED

(Camino Graveyard :: General, defect)

Description

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

We ship 2 copies of all of the table images:

1) Camino.app/Contents/MacOS/res
2) Camino.app/Contents/Resources

Is there some reason we're doing this?  The images are different, and I'm not really sure where they're used ever, so I can't tell which ones are used....

Comment 1

[Stuart Morgan]

The explicit include we do looks like it came from when we added Midas, so that's probably where they are used.

Comment 2

[Stuart Morgan]

Oh, we explicitly do both copies; once as part of the general resources copy, and one specific build phase copy. I would hazard a guess that the Resources copy was an accident--IIRC, Xcode helpfully adds things there that you put in the main resources folder.

We should just find a Midas test that uses them, rip out the Resources copy of all that stuff, and make sure it still works.

Comment 3

[Simon Fraser [no longer active]]

(In reply to comment #2)
> Oh, we explicitly do both copies; once as part of the general resources copy,
> and one specific build phase copy. I would hazard a guess that the Resources
> copy was an accident--IIRC, Xcode helpfully adds things there that you put in
> the main resources folder.

This is almost certainly the case. Gecko doesn't know anything about the Resources folder.

Comment 4

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

I'll look at kicking the project for this, then.  I remember this "feature" of Xcode being helpful from chardet. :P

Comment 5

[Stuart Morgan]

http://www.mozilla.org/editor/midasdemo/ has a demo, and you can see these images when you insert a table. I ripped out all the Resources copies and it still worked fine, which supports the mistake theory.

So all we need here is a project patch which removes the following files from the "Copy Bundle Resources" phase of the Camino and CaminoStatic targets:
grabber.gif
table-*.gif
EditorOverride.css

Comment 6

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: Trunk patch

In addition to removing the Midas crap identified by Stuart from the Bundle Resources Copy Phase (BRCP) in both targets, this

1) removes layout_xul_tree.xpt from the BRCP in both targets
2) removes libpermissions.dylib, libucvmath.dylib, and libchardet.dylib from the BRCP in the non-static target
3) removes the Wallet schema tables from the BRCP in the static target

All of these files are duplicated in various Gecko copy phases, but I ran cursory checks of XUL trees, cookie permissions, MathML, chardet, and form fill in my debug build anyway, and all was well ;)

Comment 7

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: Branch patch

In addition to removing the Midas crap identified by Stuart from the Bundle Resources Copy Phase (BRCP) in both targets, this

1) removes layout_xul_tree.xpt from the BRCP in both targets
2) removes libucvmath.dylib from the BRCP in the non-static target

All of these files are duplicated in various Gecko copy phases, but I ran cursory checks of XUL trees and MathML in my debug build anyway, and all was well ;)

Comment 8

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 245724 [details] [diff] [review]
Trunk patch

Mento, if you can't get to these quickly, can you holler, or poke another qualified Camino sr?  ;) They'll bitrot pretty quickly if we have other project changes (and some are expected, iirc).

Comment 9

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 245724 [details] [diff] [review]
Trunk patch

These have sr=smorgan per irc.

Comment 10

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 245727 [details] [diff] [review]
Branch patch

sr=smorgan per irc

Comment 11

[froodian (Ian Leue)]

Checked in on 1.8branch and trunk.