Bug 1633855 - Re-work the experimental definitions for features; r=julienw

This commit changes up the definitions of the presets a bit. I promoted the
allocations work to non-experimental, as it's fairly well supported now in the UI.
Others I demoted to experimental if they were not supported in our current UI,
or flat out don't work.

Differential Revision: https://phabricator.services.mozilla.com/D76511
This commit is contained in:
Greg Tatum 2020-05-27 22:07:08 +00:00
Родитель 680b9f7324
Коммит 873a3e02c2
2 изменённых файлов: 10 добавлений и 10 удалений

Просмотреть файл

@ -476,6 +476,8 @@ export interface FeatureDescription {
title: string, title: string,
// This will give the user a hint that it's recommended on. // This will give the user a hint that it's recommended on.
recommended?: boolean, recommended?: boolean,
// This will give the user a hint that it's an experimental feature.
experimental?: boolean,
// This will give a reason if the feature is disabled. // This will give a reason if the feature is disabled.
disabledReason?: string, disabledReason?: string,
} }

Просмотреть файл

@ -321,12 +321,6 @@ const featureDescriptions = [
"Record JavaScript stack information, and interleave it with native stacks.", "Record JavaScript stack information, and interleave it with native stacks.",
recommended: true, recommended: true,
}, },
{
name: "Responsiveness",
value: "responsiveness",
title: "Collect thread responsiveness information.",
recommended: true,
},
{ {
name: "Java", name: "Java",
value: "java", value: "java",
@ -354,6 +348,7 @@ const featureDescriptions = [
name: "Privacy", name: "Privacy",
value: "privacy", value: "privacy",
title: "Remove some potentially user-identifiable information.", title: "Remove some potentially user-identifiable information.",
experimental: true,
}, },
{ {
name: "Sequential Styling", name: "Sequential Styling",
@ -364,11 +359,13 @@ const featureDescriptions = [
name: "JIT Optimizations", name: "JIT Optimizations",
value: "trackopts", value: "trackopts",
title: "Track JIT optimizations in the JS engine.", title: "Track JIT optimizations in the JS engine.",
experimental: true,
}, },
{ {
name: "TaskTracer", name: "TaskTracer",
value: "tasktracer", value: "tasktracer",
title: "Enable TaskTracer (Experimental.)", title: "Enable TaskTracer",
experimental: true,
disabledReason: disabledReason:
"TaskTracer requires a custom build with the environment variable MOZ_TASK_TRACER set.", "TaskTracer requires a custom build with the environment variable MOZ_TASK_TRACER set.",
}, },
@ -380,7 +377,8 @@ const featureDescriptions = [
{ {
name: "JSTracer", name: "JSTracer",
value: "jstracer", value: "jstracer",
title: "Trace JS engine (Experimental.)", title: "Trace JS engine",
experimental: true,
disabledReason: disabledReason:
"JS Tracer is currently disabled due to crashes. See Bug 1565788.", "JS Tracer is currently disabled due to crashes. See Bug 1565788.",
}, },
@ -397,12 +395,12 @@ const featureDescriptions = [
{ {
name: "JS Allocations", name: "JS Allocations",
value: "jsallocations", value: "jsallocations",
title: "Track JavaScript allocations (Experimental.)", title: "Track JavaScript allocations",
}, },
{ {
name: "Native Allocations", name: "Native Allocations",
value: "nativeallocations", value: "nativeallocations",
title: "Track native allocations (Experimental.)", title: "Track native allocations",
}, },
]; ];