1585327 - Add normal-priority event queue delay time to Responsiveness measurements

Status: NEW

(Core :: Performance, enhancement, P2)

Description

[Randell Jesup [:jesup] (needinfo me)]

The Responsiveness ("jank") recorded by bug 1572337 records eventqueue responsiveness for all threads, and Input-priority responsiveness ("Jank") for PrioritizedEventQueues (aka MainThread).

It would be useful to be able to know the event queue delay for normal events on MainThread as well as for Input events. This merely requires another profiler buffertype and an argument to GetRunningEventDelay to specify which priority we care about.

Corresponding changes to the profiler webapp are needed to handle the new buffertype and display it.

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

roughly this in DoPeriodicSample():
if (is_a_prioritized_event_queue) {
aRegisteredThread.GetRunningEventDelay(currentEventDelay, currentEventRunning, EventQueuePriority::Normal);
buffer.AddEntry(ProfileBufferEntry::EventQueueDelay(EventQueuePriority::Normal, timeSinceRunningEventQueued));
}

Comment 2

[Randell Jesup [:jesup] (needinfo me)]

FYI