Closed Bug 887580 Opened 7 years ago Closed 6 years ago

files should properly export objects by attaching them directly to the global scope

Categories

(Firefox Health Report Graveyard :: Client: Desktop, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 26

People

(Reporter: smirea, Assigned: smirea)

Details

Attachments

(1 file)

Similar to Bug 887081

In order to maximize compatibility, exported objects should be attached to the global scope instead of defining them as functions (i.e. this.ProfileCreationTimeAccessor instead of function ProfileCreationTimeAccessor)
Otherwise tests fail on platforms like Boot 2 Gecko

Currently in:
services/datareporting/policy.jsm
services/healthreport/profile.jsm
Changed "profile.jsm" and "policy.jsm" to bind the exported objects to the global scope
Attachment #768078 - Flags: review?(gps)
Attachment #768078 - Flags: review?(gps) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/8105a07af658
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
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.