Closed Bug 887518 Opened 8 years ago Closed 8 years ago
Addon Manager should be loaded only when required
Currently the addon manager in the /services/healthreport/tests/xpcshell file is loaded in head.js which gets included in all tests. This causes "test_provider_addons.js" to fail when compiled on Boot2Gecko and all future tests on that platform that will call the loadAddonManager() function - which is only needed in test_provider_addons.js and therefore should only be defined there
This looks test-specific, so why 'checkin-needed' and 'sec-review' here?
Betcha this was cloned.
Moved loadAddonManager() to test_provider_addons.js which is the only file that requires it so it won't break future tests on Boot 2 Gecko
Attachment #768050 - Flags: review?(gps)
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in before you can comment on or make changes to this bug.