Bug 121137

Summary:

REGRESSION (143483): overflow:hidden doesn't quash big repaints from text-indent: -9999px

Product:

WebKit

Reporter:

Simon Fraser (smfr) <simon.fraser>

Component:

Layout and Rendering

Assignee:

Simon Fraser (smfr) <simon.fraser>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, esprehn+autocc, glenn, kondapallykalyan, rniwa, robert, simon.fraser

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Testcase	none
Patch	rniwa: review+

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