Bug 1467111 - Add the responsiveness feature to the default feature list r=mstange
authorJulien Wajsberg <felash@gmail.com>
Wed, 06 Jun 2018 15:24:08 +0200
changeset 421587 9882a1fed0d78482a861028a6afe5ebc040f2c08
parent 421586 0a06f4f9bd73fda65885fe0fc08a82ad9d5d1584
child 421588 7c04858ec3b6f297646895add59222f2d6df11e9
push id64926
push userjwajsberg@mozilla.com
push dateWed, 06 Jun 2018 14:34:20 +0000
treeherderautoland@9882a1fed0d7 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmstange
bugs1467111, 1447338
milestone62.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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");