Use friendly reason names for GC causes

RESOLVED FIXED in Firefox 42

Status

defect
RESOLVED FIXED
4 years ago
Last year

People

(Reporter: jsantell, Assigned: jsantell)

Tracking

(Blocks 1 bug)

41 Branch
Firefox 42
Dependency tree / graph

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

No description provided.
Attachment #8644555 - Flags: review?(nfitzgerald)
Comment on attachment 8644555 [details] [diff] [review]
1191153-friendly-gc.patch

Review of attachment 8644555 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/performance/test/unit/test_marker-utils.js
@@ +42,5 @@
> +  fields = Utils.getMarkerFields({ name: "GarbageCollection", cause: "ALLOC_TRIGGER" });
> +  equal(fields[0].value, "Too Many Allocations", "Uses L10N for GC reasons");
> +
> +  fields = Utils.getMarkerFields({ name: "GarbageCollection", cause: "NOT_A_GC_REASON" });
> +  equal(fields[0].value, "NOT_A_GC_REASON", "Defaults to enum for GC reasons when not L10N'd");

Good edge case for future compat, thanks!
Attachment #8644555 - Flags: review?(nfitzgerald) → review+
This landed this morning - https://hg.mozilla.org/mozilla-central/rev/a2bbe574dde2
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.