Bug 1339327 (part 1) - Fix profiler_get_features(). r=mstange.

"unwinder" and "jank" are never checked for via hasFeature(), and are not
mentioned anywhere else in the profiler. This patch removes them from
profiler_get_features().

"restyle" *does* have a hasFeature() call and is handled by
profiler_feature_active(). This patch adds it to profiler_get_features().
This commit is contained in:
Nicholas Nethercote 2017-02-15 14:26:22 +11:00
Родитель 1f1b7e1168
Коммит 7e180ba10e
1 изменённых файлов: 3 добавлений и 8 удалений

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

@ -1912,16 +1912,9 @@ profiler_get_features()
// Include the C++ leaf node if not stackwalking. DevTools
// profiler doesn't want the native addresses.
"leaf",
#endif
#if !defined(SPS_OS_windows)
// Use a seperate thread of walking the stack.
"unwinder",
#endif
"java",
// Only record samples during periods of bad responsiveness
"jank",
// Tell the JS engine to emmit pseudostack entries in the
// pro/epilogue.
// Tell the JS engine to emit pseudostack entries in the prologue/epilogue.
"js",
// GPU Profiling (may not be supported by the GL)
"gpu",
@ -1937,6 +1930,8 @@ profiler_get_features()
"mainthreadio",
// Add RSS collection
"memory",
// Restyle profiling.
"restyle",
#ifdef MOZ_TASK_TRACER
// Start profiling with feature TaskTracer.
"tasktracer",