Bug 1024063 - Make the 'threads' feature implicit if thread filter is specific. r=ehsan

This commit is contained in:
Benoit Girard 2014-06-11 15:53:50 -04:00
parent 114ff5488a
commit 95cea05798

View File

@ -65,7 +65,9 @@ class TableTicker: public Sampler {
mProfileJS = hasFeature(aFeatures, aFeatureCount, "js");
mProfileJava = hasFeature(aFeatures, aFeatureCount, "java");
mProfilePower = hasFeature(aFeatures, aFeatureCount, "power");
mProfileThreads = hasFeature(aFeatures, aFeatureCount, "threads");
// Users sometimes ask to filter by a list of threads but forget to request
// profiling non main threads. Let's make it implificit if we have a filter
mProfileThreads = hasFeature(aFeatures, aFeatureCount, "threads") || aFilterCount > 0;
mUnwinderThread = hasFeature(aFeatures, aFeatureCount, "unwinder") || sps_version2();
mAddLeafAddresses = hasFeature(aFeatures, aFeatureCount, "leaf");
mPrivacyMode = hasFeature(aFeatures, aFeatureCount, "privacy");