Bug 57013

Summary: Web Inspector: Highlight visible lines first
Product: WebKit Reporter: Andrey Adaikin <aandrey>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, apavlov, bweinstein, eric, joepeck, keishi, loislo, pfeldman, pmuellr, podivilov, rik, timothy, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Andrey Adaikin
Reported 2011-03-24 07:20:43 PDT
We should highlight visible lines first, so that in case of a source with long lines we should not wait while invisible lines in an expanded chunk are being highlighted. This delay may be very noticeable.
Attachments
Patch (8.18 KB, patch)
2011-03-24 08:27 PDT, Andrey Adaikin 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andrey Adaikin
Comment 1 2011-03-24 08:27:47 PDT
Created attachment 86778 [details] Patch
Pavel Podivilov
Comment 2 2011-03-29 04:20:31 PDT
Comment on attachment 86778 [details] Patch Clearing flags on attachment: 86778 Committed r82227: <http://trac.webkit.org/changeset/82227>
Pavel Podivilov
Comment 3 2011-03-29 04:20:40 PDT
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 4 2011-03-29 05:34:41 PDT
http://trac.webkit.org/changeset/82227 might have broken GTK Linux 64-bit Debug The following tests are not passing: svg/text/text-deco-01-b.svg
Note You need to log in before you can comment on or make changes to this bug.