RESOLVED FIXED 157612
Web Inspector: 11% of time in TimelineRecording spent updating DataGrid that is not visible
https://bugs.webkit.org/show_bug.cgi?id=157612
Summary Web Inspector: 11% of time in TimelineRecording spent updating DataGrid that ...
Joseph Pecoraro
Reported 2016-05-11 22:45:51 PDT
* SUMMARY 11% of time in TimelineRecording spent updating DataGrid that is not visible DataGrid._updateVisibleRows is taking up 11% of time, including many forced layouts, when the DataGrid is not even visible. In this case, the TimelineRecordingProgressView is showing, but in other cases, it likely isn't updating unless there is some kind of resize. * STEPS TO REPRODUCE 1. Inspect theverge.com 2. Show Timeline tab 3. Show Scripts > Call Trees 4. Start Recording => Lots of time and forced layouts in DataGrid._updateVisibleRows
Attachments
[Patch] Proposed Fix (3.46 KB, patch)
2016-05-12 04:08 PDT, Matt Baker
no flags
[Patch] Proposed Fix (4.19 KB, patch)
2016-05-12 09:52 PDT, Matt Baker
no flags
Radar WebKit Bug Importer
Comment 1 2016-05-11 22:46:08 PDT
Matt Baker
Comment 2 2016-05-12 04:08:01 PDT
Created attachment 278728 [details] [Patch] Proposed Fix
Timothy Hatcher
Comment 3 2016-05-12 08:28:00 PDT
Comment on attachment 278728 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=278728&action=review > Source/WebInspectorUI/UserInterface/Views/TimelineView.js:312 > + if (WebInspector.timelineManager.isCapturing() && !this.showsLiveRecordingData) The ChangeLog says if the view supports live, but this does !live. Which is right?
Matt Baker
Comment 4 2016-05-12 09:45:16 PDT
Comment on attachment 278728 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=278728&action=review >> Source/WebInspectorUI/UserInterface/Views/TimelineView.js:312 >> + if (WebInspector.timelineManager.isCapturing() && !this.showsLiveRecordingData) > > The ChangeLog says if the view supports live, but this does !live. Which is right? The ChangeLog is correct. Copy paste error.
Matt Baker
Comment 5 2016-05-12 09:52:41 PDT
Created attachment 278736 [details] [Patch] Proposed Fix
WebKit Commit Bot
Comment 6 2016-05-12 10:56:42 PDT
Comment on attachment 278736 [details] [Patch] Proposed Fix Clearing flags on attachment: 278736 Committed r200779: <http://trac.webkit.org/changeset/200779>
WebKit Commit Bot
Comment 7 2016-05-12 10:56:47 PDT
All reviewed patches have been landed. Closing bug.
Joseph Pecoraro
Comment 8 2016-05-12 12:33:59 PDT
This is great! But there is still a concern that when a datagrid is showing (say layout and rendering) the datagrid is constantly forcing layouts. I wonder if we can improve _updateVisibleRows to not do as much work when its constraints are not changing. For example the view is not resizing and not scrolling, it should probably not always force a layout.
Note You need to log in before you can comment on or make changes to this bug.