Closed Bug 838461 Opened 12 years ago Closed 12 years ago

Bagheera client failing after JSM merging

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 21

People

(Reporter: gps, Assigned: gps)

References

Details

Attachments

(1 file)

Due to bug 838458, when we merged JSMs in bug 836177 we accidentally broke the Bagheera HTTP client. I doubt Nightly has been submitting FHR payloads since Jan 31 because of this. #fail. Patch forthcoming.
The main patch is moving the rest.js inclusion. rest.js is now always Cu.imported because it isn't compatible with strict mode. This will regress memory usage slightly until we make rest.js work in strict mode. The rest of the patch is to make RESTRequest log into BagheeraClient. I've been wanting to do this for a while and this was as good of an excuse as any.
Attachment #710488 - Flags: review?(rnewman)
Obligatory notice to njn that this may regress heap size by compartment overhead amount until bug 838458 is resolved or until zones lands.
Attachment #710488 - Flags: review?(rnewman) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in services]
Target Milestone: --- → mozilla21
Component: Metrics and Firefox Health Report → Client: Desktop
Product: Mozilla Services → Firefox Health Report
Target Milestone: mozilla21 → Firefox 21
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: