Bug 1467111 - Add the responsiveness feature to the default feature list r=mstange draft
authorJulien Wajsberg <felash@gmail.com>
Wed, 06 Jun 2018 15:24:08 +0200
changeset 804732 362a183c998ae435a4b4374aa6a40cf806b3d44e
parent 804634 cec4a3cecc29ff97860198969b6fdff24b9e93bb
push id112453
push userbmo:felash@gmail.com
push dateWed, 06 Jun 2018 14:24:21 +0000
reviewersmstange
bugs1467111, 1447338
milestone62.0a1
Bug 1467111 - Add the responsiveness feature to the default feature list r=mstange Bug 1447338 introduced the new feature for responsiveness handling, but we forgot to include it in the default feature list. As a result it's not included by default when profiling the startup. MozReview-Commit-ID: 1YPcDaGZR9s
tools/profiler/core/platform.cpp
--- a/tools/profiler/core/platform.cpp
+++ b/tools/profiler/core/platform.cpp
@@ -2459,16 +2459,17 @@ profiler_init(void* aStackTop)
                       ProfilerFeature::Java |
 #endif
                       ProfilerFeature::JS |
                       ProfilerFeature::Leaf |
 #if defined(HAVE_NATIVE_UNWIND)
                       ProfilerFeature::StackWalk |
 #endif
                       ProfilerFeature::Threads |
+                      ProfilerFeature::Responsiveness |
                       0;
 
   UniquePtr<char[]> filterStorage;
 
   nsTArray<const char*> filters;
   filters.AppendElement("GeckoMain");
   filters.AppendElement("Compositor");
   filters.AppendElement("DOM Worker");