diff options
author | Moonchild <mcwerewolf@wolfbeast.com> | 2019-01-14 20:21:16 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-01-14 20:21:16 +0100 |
commit | fb1d4c9712eb1898d89f90edfc6120ff169e6357 (patch) | |
tree | 677d602abee695200146c4d51ba8db10146bc38c /toolkit/components/perfmonitoring | |
parent | f38edc94a31de3bae839cf63ed57c3851908ac46 (diff) | |
parent | 6335404642a019df481275f4f290ea76b4d8f597 (diff) | |
download | UXP-fb1d4c9712eb1898d89f90edfc6120ff169e6357.tar UXP-fb1d4c9712eb1898d89f90edfc6120ff169e6357.tar.gz UXP-fb1d4c9712eb1898d89f90edfc6120ff169e6357.tar.lz UXP-fb1d4c9712eb1898d89f90edfc6120ff169e6357.tar.xz UXP-fb1d4c9712eb1898d89f90edfc6120ff169e6357.zip |
Merge pull request #929 from adeshkp/remove-telemetry-stubs
Telemetry: Remove stubs and related code
Diffstat (limited to 'toolkit/components/perfmonitoring')
-rw-r--r-- | toolkit/components/perfmonitoring/nsPerformanceStats.cpp | 16 | ||||
-rw-r--r-- | toolkit/components/perfmonitoring/nsPerformanceStats.h | 15 |
2 files changed, 0 insertions, 31 deletions
diff --git a/toolkit/components/perfmonitoring/nsPerformanceStats.cpp b/toolkit/components/perfmonitoring/nsPerformanceStats.cpp index 33aeaf7c2..6c470356a 100644 --- a/toolkit/components/perfmonitoring/nsPerformanceStats.cpp +++ b/toolkit/components/perfmonitoring/nsPerformanceStats.cpp @@ -662,9 +662,6 @@ nsPerformanceStatsService::nsPerformanceStatsService() nsPerformanceGroup::GroupScope::RUNTIME // scope )) , mIsHandlingUserInput(false) - , mProcessStayed(0) - , mProcessMoved(0) - , mProcessUpdateCounter(0) , mIsMonitoringPerCompartment(false) , mJankAlertThreshold(mozilla::MaxValue<uint64_t>::value) // By default, no alerts , mJankAlertBufferingDelay(1000 /* ms */) @@ -953,13 +950,6 @@ nsPerformanceStatsService::SetJankAlertBufferingDelay(uint32_t value) { return NS_OK; } -nsresult -nsPerformanceStatsService::UpdateTelemetry() -{ - /* STUB */ - return NS_OK; -} - /* static */ nsIPerformanceStats* nsPerformanceStatsService::GetStatsForGroup(const js::PerformanceGroup* group) @@ -992,12 +982,6 @@ nsPerformanceStatsService::GetSnapshot(JSContext* cx, nsIPerformanceSnapshot * * } } - js::GetPerfMonitoringTestCpuRescheduling(cx, &mProcessStayed, &mProcessMoved); - - if (++mProcessUpdateCounter % 10 == 0) { - mozilla::Unused << UpdateTelemetry(); - } - snapshot.forget(aSnapshot); return NS_OK; diff --git a/toolkit/components/perfmonitoring/nsPerformanceStats.h b/toolkit/components/perfmonitoring/nsPerformanceStats.h index c82a3e92c..6902c840d 100644 --- a/toolkit/components/perfmonitoring/nsPerformanceStats.h +++ b/toolkit/components/perfmonitoring/nsPerformanceStats.h @@ -363,21 +363,6 @@ protected: nsPerformanceGroup* group); - - - /********************************************************** - * - * To check whether our algorithm makes sense, we keep count of the - * number of times the process has been rescheduled to another CPU - * while we were monitoring the performance of a group and we upload - * this data through Telemetry. - */ - nsresult UpdateTelemetry(); - - uint64_t mProcessStayed; - uint64_t mProcessMoved; - uint32_t mProcessUpdateCounter; - /********************************************************** * * Options controlling measurements. |