Closed Bug 703523 Opened 13 years ago Closed 13 years ago

#ifdef windows and crash reporter specific parts of browser.js

Categories

(Firefox :: General, defect)

All
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 13

People

(Reporter: glandium, Assigned: glandium)

Details

Attachments

(1 file, 1 obsolete file)

I wrote a script that looks for resource:// uris in the built code and checks whether the corresponding files exist, so as to detect problems with resource://gre/ used where resource:/// was meant. It turns out it also detects "problems" with imports of files that are not installed in every cases. For instance, in browser.js, a file that is only installed in windows builds is imported, as well as a file that is only installed when the crash reporter is built. While these are cosmetic issues, it would make my life easier if the patch i'm about to attach would be applied.
Attachment #575411 - Attachment is obsolete: true
Attachment #597767 - Flags: review?(gavin.sharp)
Attachment #597767 - Flags: review?(gavin.sharp) → review+
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 13
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: