RESOLVED FIXED153390
Web Inspector: Reduce unnecessary forced layouts in TimelineRuler 
https://bugs.webkit.org/show_bug.cgi?id=153390
Summary Web Inspector: Reduce unnecessary forced layouts in TimelineRuler
Joseph Pecoraro
Reported 2016-01-22 21:18:18 PST
* SUMMARY Reduce unnecessary forced layouts in TimelineRuler TimelineRuler is potentially forcing a synchronous layout (element.clientWidth) every time it does a layout. This is unnecessary expensive, and in some cases this causes the entire inspector process to hang while it draws/layout/draw/layout repeatedly. The only reason the TimelineRuler would need its clientWidth is to calculate the width of the element. So this value can be updated only when the ruler is resized.
Attachments
[PATCH] Proposed Fix (7.94 KB, patch)
2016-01-22 21:23 PST, Joseph Pecoraro 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2016-01-22 21:18:32 PST
<rdar://problem/24312241>
Joseph Pecoraro
Comment 2 2016-01-22 21:23:31 PST
Created attachment 269639 [details] [PATCH] Proposed Fix
WebKit Commit Bot
Comment 3 2016-01-22 22:47:08 PST
Comment on attachment 269639 [details] [PATCH] Proposed Fix Clearing flags on attachment: 269639 Committed r195504: <http://trac.webkit.org/changeset/195504>
WebKit Commit Bot
Comment 4 2016-01-22 22:47:12 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.