Bug 116344 - Ads on theverge.com cause repaints when hovered, even though content doesn't visibly change.
Summary: Ads on theverge.com cause repaints when hovered, even though content doesn't ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Andreas Kling
URL:
Keywords: Performance
Depends on:
Blocks:
 
Reported: 2013-05-17 13:04 PDT by Andreas Kling
Modified: 2013-05-17 19:55 PDT (History)
4 users (show)

See Also:


Attachments
Proposed patch (3.22 KB, patch)
2013-05-17 13:05 PDT, Andreas Kling
darin: review+
Details | Formatted Diff | Diff
Latch for panding (3.22 KB, patch)
2013-05-17 14:05 PDT, Andreas Kling
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.