Bug 103802

Summary: Don't call repaintBlockSelectionGaps() after layout if there was never a selection
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: ahmad.saleem792, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   

Simon Fraser (smfr)
Reported 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
Attachments
Add attachment proposed patch, testcase, etc.
Ahmad Saleem
Comment 1 2022-11-26 16:20:23 PST
Hi Simon - is it something we needed now? Thanks!
Note You need to log in before you can comment on or make changes to this bug.