Closed Bug 836285 Opened 7 years ago Closed Last year
Unhack JSM concatenation in /services
In bug 834936 and bug 836177 we used preprocessor magic to concatenate JSMs together to load them in a single compartment so we wouldn't have so much memory overhead from multiple compartments. This is ugly and prone to subtle gotchas. As soon as zones lands we should unhack these changes (after veriyfing we don't regress memory too much, of course).
Component: Metrics and Firefox Health Report → Client: Desktop
Product: Mozilla Services → Firefox Health Report
Zones have landed now.
At various times I've attempted to unhack the concatenation. However, there are some subtle "namespacing" issues w.r.t. providers and their prototypes that causes some things to fail pretty hard. I think the same type is getting imported under two names and the prototype chain checking is failing as a result. Whoever takes this bug will have a "fun" time sorting it out!
FHR is going away per bug 1209088, lets take a look whether this is still needed.
See https://bugzilla.mozilla.org/show_bug.cgi?id=1497137; component has been deprecated.
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.