Closed Bug 748627 Opened 12 years ago Closed 12 years ago

Bug 741378 broke MEMORY_JS_GC_HEAP telemetry

Categories

(Toolkit :: Telemetry, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla14
Tracking Status
firefox14 + fixed
firefox15 - ---

People

(Reporter: justin.lebar+bug, Assigned: justin.lebar+bug)

References

Details

(Keywords: regression)

Attachments

(1 file)

Bug 741378 moved the js-gc-heap memory reporter into a multi-reporter, which isn't supported by telemetry.

I have a fix for this in bug 748440.  For FF14, we should back out bug 741378 from Aurora so we don't have a big gap in the data.
Assignee: nobody → justin.lebar+bug
Attached patch BackoutSplinter Review
Comment on attachment 618130 [details] [diff] [review]
Backout

[Approval Request Comment]
Regression caused by (bug #): bug 741378

User impact if declined: None; affects telemetry only.

Testing completed (on m-c, etc.): None.

Risk to taking this patch (and alternatives if risky): Backout to known good state.  But if we did not take this patch, the only affect would be on telemetry data collection; it's not the end of the world.

String changes made by this patch: None.
Attachment #618130 - Flags: approval-mozilla-aurora?
Comment on attachment 618130 [details] [diff] [review]
Backout

[Triage comment]
Please go ahead and land on Aurora (Firefox 14)
Attachment #618130 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Fixed in Aurora for FF 14.

https://hg.mozilla.org/releases/mozilla-aurora/rev/7525533b3e42

I'm going to leave this bug open until we fix the issue in trunk (bug 748440).
Target Milestone: --- → mozilla14
Justin - Now that bug 748440 has been resolved can this bug be resolved as well?
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: