Bug 107649 - Web Inspector: remove invisibleLineRow code from paintLines DTE method
Summary: Web Inspector: remove invisibleLineRow code from paintLines DTE method
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Andrey Lushnikov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-23 01:55 PST by Andrey Lushnikov
Modified: 2013-01-23 09:47 PST (History)
9 users (show)

See Also:

Attachments
Patch (2.67 KB, patch)
2013-01-23 01:58 PST, Andrey Lushnikov 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Lushnikov 2013-01-23 01:55:29 PST 
As _paintLines method does everything synchronously, the order of linerows being processed does not matter. Removing "invisibleLineRow" logic will eliminate calls to scrollTop and offsetTop methods (which force sync relayout).
Comment 1 Andrey Lushnikov 2013-01-23 01:58:41 PST 
Created attachment 184182 [details]
Patch
Comment 2 WebKit Review Bot 2013-01-23 09:47:54 PST 
Comment on attachment 184182 [details]
Patch

Clearing flags on attachment: 184182

Committed r140538: <http://trac.webkit.org/changeset/140538>
Comment 3 WebKit Review Bot 2013-01-23 09:47:57 PST 
All reviewed patches have been landed.  Closing bug.