Bug 103802 - Don't call repaintBlockSelectionGaps() after layout if there was never a selection
Summary: Don't call repaintBlockSelectionGaps() after layout if there was never a sele...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-30 20:48 PST by Simon Fraser (smfr)
Modified: 2022-11-26 16:20 PST (History)
2 users (show)

See Also:

Attachments
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 Simon Fraser (smfr) 2012-11-30 20:48:56 PST 
When scrolling a long page, we spend a small amount of time under FrameSelection::updateAppearance(), calling clearSelection() which runs through all layers calling RenderLayer::repaintBlockSelectionGaps(). This is just a waste of cycles if there never was a selection.

I was using a long review page with comments visible, on theverge.com
Comment 1 Ahmad Saleem 2022-11-26 16:20:23 PST 
Hi Simon - is it something we needed now? Thanks!