Bug 151058

Summary: Web Inspector: Reduce synchronous view layouts
Product: WebKit Reporter: Matt Baker <mattbaker>
Component: Web InspectorAssignee: Matt Baker <mattbaker>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Bug Depends on: 150993    
Bug Blocks: 146256    
Attachments:
Description Flags
[Patch] Proposed Fix none

Matt Baker
Reported 2015-11-09 15:51:16 PST
* SUMMARY Reduce synchronous view layouts. We should be calling View.needsLayout rather than View.updateLayout, unless an up-to-date attribute value for a DOM element is needed. * NOTES Once https://bugs.webkit.org/show_bug.cgi?id=150993 is fixed, we should see the majority of view updates occur on the main animation frame. If not, it means that synchronous layouts are flushing the layout tree before the animation frame fires. Switching to asynchronous layouts should resolve cases where redundant layouts occur (see https://bugs.webkit.org/show_bug.cgi?id=146256).
Attachments
[Patch] Proposed Fix (11.25 KB, patch)
2015-11-18 16:06 PST, Matt Baker 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2015-11-09 15:52:00 PST
<rdar://problem/23472397>
Matt Baker
Comment 2 2015-11-18 16:06:19 PST
Created attachment 265798 [details] [Patch] Proposed Fix
WebKit Commit Bot
Comment 3 2015-11-18 21:44:37 PST
Comment on attachment 265798 [details] [Patch] Proposed Fix Clearing flags on attachment: 265798 Committed r192612: <http://trac.webkit.org/changeset/192612>
WebKit Commit Bot
Comment 4 2015-11-18 21:44:41 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.