Bug 121137 - REGRESSION (143483): overflow:hidden doesn't quash big repaints from text-indent: -9999px
Summary: REGRESSION (143483): overflow:hidden doesn't quash big repaints from text-ind...
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: Simon Fraser (smfr)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2013-09-10 17:57 PDT by Simon Fraser (smfr)
Modified: 2013-09-11 11:32 PDT (History)
7 users (show)

See Also:

Attachments
Testcase (843 bytes, text/html)
2013-09-10 17:57 PDT, Simon Fraser (smfr) 		no flags Details
Patch (4.78 KB, patch)
2013-09-10 18:25 PDT, Simon Fraser (smfr) 		rniwa: review+
Details | Formatted Diff | Diff
View All 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) 2013-09-10 17:57:04 PDT 
Load the attachment in Minibrowser (WK1), launch QuartzDebug, and turn on "Flash screen updates". Now hover over the gradient bit. Note how the repaint extents outside the left of the box.

The box has overflow:hidden, so why is the repaint so big?

This kills our advice to combine text-indent: -9999px with overflow:hidden
Comment 1 Simon Fraser (smfr) 2013-09-10 17:57:19 PDT 
Created attachment 211271 [details]
Testcase
Comment 2 Simon Fraser (smfr) 2013-09-10 18:25:39 PDT 
Created attachment 211273 [details]
Patch
Comment 3 Simon Fraser (smfr) 2013-09-10 18:28:53 PDT 
<rdar://problem/14030862>
Comment 4 Ryosuke Niwa 2013-09-10 18:40:08 PDT 
Maybe http://trac.webkit.org/changeset/143313 fixed the original bug?
Comment 5 Simon Fraser (smfr) 2013-09-11 11:32:31 PDT 
https://trac.webkit.org/r155546