[Webkit-unassigned] [Bug 195789] New: Web Inspector: CPU Usage Timeline - the right edge of each column should align with a CPU measurement

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=195789

            Bug ID: 195789
           Summary: Web Inspector: CPU Usage Timeline - the right edge of
                    each column should align with a CPU measurement
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: All
                OS: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Web Inspector
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: drousso at apple.com
                CC: inspector-bugzilla-changes at group.apple.com

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

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20190315/353f8a72/attachment.html>