Bug 1666975 - Remove PENDING_CRITICAL_INPUT_WHEN_TIMEOUT telemetry probe r=smaug

This probe is no longer needed.

Differential Revision: https://phabricator.services.mozilla.com/D91456
This commit is contained in:
Sean Feng 2020-09-25 20:57:51 +00:00
Родитель 9aa3b6a184
Коммит f10ed9ceaa
3 изменённых файлов: 0 добавлений и 36 удалений

Просмотреть файл

@ -400,27 +400,6 @@ void TimeoutManager::UpdateBudget(const TimeStamp& aNow,
mLastBudgetUpdate = aNow;
}
size_t TimeoutManager::GetNumPendingInputs() {
ContentChild* contentChild = ContentChild::GetSingleton();
mozilla::ipc::MessageChannel* channel =
contentChild ? contentChild->GetIPCChannel() : nullptr;
if (channel) {
size_t count = 0;
channel->PeekMessages([&count](const IPC::Message& aMsg) -> bool {
if (nsContentUtils::IsMessageCriticalInputEvent(aMsg)) {
// The max number we can record in the telemetry is 80,
// so we don't need to continue the counting.
if (++count > 80) {
return false;
}
}
return true;
});
return count;
}
return 0;
}
// The longest interval (as PRIntervalTime) we permit, or that our
// timer code can handle, really. See DELAY_INTERVAL_LIMIT in
// nsTimerImpl.h for details.
@ -911,8 +890,6 @@ void TimeoutManager::RunTimeout(const TimeStamp& aNow,
mLastFiringIndex = timeout->mFiringIndex;
#endif
// This timeout is good to run.
Telemetry::Accumulate(Telemetry::PENDING_CRITICAL_INPUT_WHEN_TIMEOUT,
GetNumPendingInputs());
bool timeout_was_cleared = window->RunTimeoutHandler(timeout, scx);
#if MOZ_GECKO_PROFILER
if (profiler_can_accept_markers()) {

Просмотреть файл

@ -103,8 +103,6 @@ class TimeoutManager final {
void SetLoading(bool value);
size_t GetNumPendingInputs();
private:
void MaybeStartThrottleTimeout();

Просмотреть файл

@ -1677,17 +1677,6 @@
"description": "Time between receiving a scroll event on the event loop and compositing its result onto the screen (ms)",
"bug_numbers": [1500465, 1604818]
},
"PENDING_CRITICAL_INPUT_WHEN_TIMEOUT": {
"record_in_processes": ["content"],
"products": ["firefox"],
"alert_emails": ["perfteam@mozilla.com", "seanfeng@mozilla.com"],
"expires_in_version": "84",
"kind": "enumerated",
"n_values": 80,
"releaseChannelCollection": "opt-out",
"description": "Measures the number of pending critical input events in the IPC channel when the timeout handler is about to run",
"bug_numbers": [1648284]
},
"CANVAS_2D_USED": {
"record_in_processes": ["main", "content"],
"products": ["firefox", "fennec"],