Bug 195789

Summary:

Web Inspector: CPU Usage Timeline - the right edge of each column should align with a CPU measurement

Product:

WebKit

Reporter:

Devin Rousso <hi>

Component:

Web Inspector

Assignee:

Devin Rousso <hi>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, hi, inspector-bugzilla-changes, joepeck, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	none
Patch	none

Devin Rousso

Reported 2019-03-14 20:06:22 PDT

Right now, we are rendering each column such that the middle of the column is aligned with the CPU measurement. This could potentially be misleading, as due to the width of the bar, it implies that there was a period of time after the actual CPU measurement that is attributed to that same CPU measurement. 1 2 3 _______ [ * ] [ * ]_______ [ * ][ * ] _______[ * ][ * ] [ * ][ * ][ * ] [__*__][__*__][__*__] A B C D E F In this example, a developer might "attribute" any work done at time B to record 1, when in reality it should be "attributed" to record 2, since the CPU measurement had already been taken by the time B was captured, meaning that the work for B hadn't yet been don and could therefore not have affected the CPU measurement We should be rendering the columns such that the CPU measurement aligns with the trailing edge of the column. 1 2 3 _______ ___ [ *] [ [ *]_______[ [ *][ *][ ____[ *][ *][ *][ *][ *][ __*][____*][____*][__ A B C D E F NOT: the drawing isn't exactly accurate, as the * would overlap the ]. Legend: [ ] = a column for a CPU measurement (500ms wide to account for the fact that CPU measurements are 500ms apart) * = the time when the measurement actually takes place

Attachments
Patch (3.96 KB, patch) 2019-03-14 22:04 PDT, Devin Rousso	no flags	Details Formatted Diff Diff
Patch (6.19 KB, patch) 2019-03-15 20:49 PDT, Devin Rousso	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.