Closed Bug 820546 Opened 9 years ago Closed 9 years ago
Files are preprocessed unnecessary when building fennec's chrome jar
The jar manifest at http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/jar.mn specifies that some files be preprocessed (the ones with a '*' in the first column). However some of those files don't need to be preprocessed. Specifically, downloads.js and browser.xul do not appear to be modified during preprocessing.
A couple of other files also don't need preprocessing if they use XML comments instead of # comments. Will request review once this is done building for me and I can test it.
Assignee: nobody → bugmail.mozilla
Whoops, attached the wrong patch in the last comment. Here is the right one, and it does seem to work fine
Comment on attachment 691076 [details] [diff] [review] Patch In the old days, we used preprocessor comments, rather than HTML comments, because the resulting code file was smaller and had less to parse. I'm not sure that ever helped too much, and probably makes no difference now. So, sure... let's do this.
Attachment #691076 - Flags: review?(mark.finkle) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.