Closed
Bug 748627
Opened 12 years ago
Closed 12 years ago
Bug 741378 broke MEMORY_JS_GC_HEAP telemetry
Categories
(Toolkit :: Telemetry, defect)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: justin.lebar+bug, Assigned: justin.lebar+bug)
References
Details
(Keywords: regression)
Attachments
(1 file)
17.93 KB,
patch
|
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
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 | ||
Updated•12 years ago
|
Assignee: nobody → justin.lebar+bug
Assignee | ||
Updated•12 years ago
|
status-firefox14:
--- → affected
status-firefox15:
--- → affected
tracking-firefox14:
--- → ?
tracking-firefox15:
--- → ?
Keywords: regression
Assignee | ||
Comment 1•12 years ago
|
||
Assignee | ||
Comment 2•12 years ago
|
||
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 3•12 years ago
|
||
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+
Updated•12 years ago
|
Assignee | ||
Comment 4•12 years ago
|
||
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
Updated•12 years ago
|
status-firefox15:
affected → ---
Comment 5•12 years ago
|
||
Justin - Now that bug 748440 has been resolved can this bug be resolved as well?
Assignee | ||
Updated•12 years ago
|
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.
Description
•