Bug 135371

Summary: Web Inspector: Incorrectly sized TimelineDataGrid event bubble
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[IMAGE] Issue
none
[IMAGE] Expected
none
[PATCH] Proposed Fix none

Description Joseph Pecoraro 2014-07-28 16:57:12 PDT 
* SUMMARY
Sometimes there are incorrectly sized TimelineDataGrid event bubbles in the overview view. This is misleading. See attached screenshots.

* STEPS TO REPRODUCE
1. Inspect daringfireball.net
2. Click Timelines
3. Select a time range of about 0-200ms
4. Reload the page (should show some stuff, but there should be JS evaluation after the current selection range)
5. Select a time range of about 300ms-2000ms
  => script bubble is much larger than expected
Comment 1 Radar WebKit Bug Importer 2014-07-28 16:57:38 PDT 
<rdar://problem/17836658>
Comment 2 Joseph Pecoraro 2014-07-28 17:00:55 PDT 
Created attachment 235645 [details]
[IMAGE] Issue
Comment 3 Joseph Pecoraro 2014-07-28 17:01:14 PDT 
Created attachment 235646 [details]
[IMAGE] Expected
Comment 4 Joseph Pecoraro 2014-07-28 19:33:55 PDT 
This is happening because in TimelineDataGridNode.prototype.refreshGraph the visibleWidth is 0. That is scary, because it will almost always happen before elements are added at the graph refreshed.

It is just using that to calculate the secondsPerPixel, including doing an offsetWidth per-datagridnode. We can eliminate this possible forced layout, as well as the calculation because the graph data source must already know the secondsPerPixel! Patch coming.
Comment 5 Joseph Pecoraro 2014-07-28 19:41:26 PDT 
Created attachment 235657 [details]
[PATCH] Proposed Fix
Comment 6 Timothy Hatcher 2014-07-28 20:45:55 PDT 
Comment on attachment 235657 [details]
[PATCH] Proposed Fix

Awesome!
Comment 7 WebKit Commit Bot 2014-07-28 21:18:05 PDT 
Comment on attachment 235657 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 235657

Committed r171721: <http://trac.webkit.org/changeset/171721>
Comment 8 WebKit Commit Bot 2014-07-28 21:18:08 PDT 
All reviewed patches have been landed.  Closing bug.