Backed out changeset 01dac856fde0 (bug 1198196) for mulet problems

This commit is contained in:
Carsten "Tomcat" Book 2015-11-13 13:39:35 +01:00
parent 31c87cfffa
commit 0e54e274a6
2 changed files with 12 additions and 5 deletions

View File

@ -4244,13 +4244,14 @@
"releaseChannelCollection": "opt-out",
"description": "Whether the homepage was imported during browser migration. Only available on release builds during firstrun."
},
"EVENTLOOP_UI_ACTIVITY_EXP_MS": {
"EVENTLOOP_UI_LAG_EXP_MS": {
"alert_emails": ["perf-telemetry-alerts@mozilla.com"],
"bug_numbers": [1198196],
"expires_in_version": "never",
"kind": "exponential",
"low": 50,
"high": "60000",
"n_buckets": 50,
"n_buckets": 20,
"extended_statistics_ok": true,
"description": "Widget: Time it takes for the message before a UI message (ms)"
},
"FX_SESSION_RESTORE_STARTUP_INIT_SESSION_MS": {

View File

@ -370,10 +370,16 @@ NotifyActivity(ActivityType aActivityType)
// penalties here.
gTimestamp = PR_IntervalNow();
// If we have UI activity we should reset the timer and report it
// If we have UI activity we should reset the timer and report it if it is
// significant enough.
if (aActivityType == kUIActivity) {
mozilla::Telemetry::Accumulate(mozilla::Telemetry::EVENTLOOP_UI_ACTIVITY_EXP_MS,
// The minimum amount of lag time that we should report for telemetry data.
// Mozilla's UI responsiveness goal is 50ms
static const uint32_t kUIResponsivenessThresholdMS = 50;
if (cumulativeUILagMS > kUIResponsivenessThresholdMS) {
mozilla::Telemetry::Accumulate(mozilla::Telemetry::EVENTLOOP_UI_LAG_EXP_MS,
cumulativeUILagMS);
}
cumulativeUILagMS = 0;
}