Bug 116344

Summary: Ads on theverge.com cause repaints when hovered, even though content doesn't visibly change.
Product: WebKit Reporter: Andreas Kling <kling>
Component: Layout and RenderingAssignee: Andreas Kling <kling>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, koivisto
Priority: P2 Keywords: Performance
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Proposed patch
darin: review+
Latch for panding none

Description Andreas Kling 2013-05-17 13:04:27 PDT 
There's a :hover selector that changes the outline value, but the both before/after computed style has non-visible outlines.
We should be smart enough to know that there's no repaint necessary.
Comment 1 Andreas Kling 2013-05-17 13:05:11 PDT 
Created attachment 202140 [details]
Proposed patch
Comment 2 Andreas Kling 2013-05-17 14:05:00 PDT 
Created attachment 202155 [details]
Latch for panding
Comment 3 WebKit Commit Bot 2013-05-17 19:54:58 PDT 
The commit-queue encountered the following flaky tests while processing attachment 202155 [details]:

svg/batik/text/textLength.svg bug 116329 (authors: darin@apple.com and zimmermann@kde.org)
svg/batik/text/textPosition.svg bug 116328 (authors: darin@apple.com and zimmermann@kde.org)
svg/batik/paints/patternRegions.svg bug 116380 (author: zimmermann@kde.org)
media/track/track-remove-crash.html bug 115892 (author: eric.carlson@apple.com)
fast/js/post-message-numeric-property.html bug 116381 (author: fpizlo@apple.com)
The commit-queue is continuing to process your patch.
Comment 4 WebKit Commit Bot 2013-05-17 19:55:49 PDT 
Comment on attachment 202155 [details]
Latch for panding

Clearing flags on attachment: 202155

Committed r150318: <http://trac.webkit.org/changeset/150318>
Comment 5 WebKit Commit Bot 2013-05-17 19:55:51 PDT 
All reviewed patches have been landed.  Closing bug.