Bug 124651

Summary:

Don't paint simple text runs outside the paint rect

Product:

WebKit

Reporter:

Antti Koivisto <koivisto>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, esprehn+autocc, glenn, kondapallykalyan

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
patch	andersca: review+

Description Antti Koivisto 2013-11-20 03:27:06 PST

optimize

Comment 1 Antti Koivisto 2013-11-20 03:43:51 PST

Created attachment 217416 [details]
patch

Comment 2 Anders Carlsson 2013-11-20 06:53:19 PST

Comment on attachment 217416 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=217416&action=review

> Source/WebCore/rendering/SimpleLineLayoutFunctions.cpp:90
>          auto run = *it;

I think this copies the run. Maybe it should be const auto& instead?

Comment 3 Antti Koivisto 2013-11-20 07:12:06 PST

https://trac.webkit.org/r159560