Bug 78249

Summary:

[Chromium] TiledLayerChromium::protectVisibleTileTextures() should only protect the visible textures

Product:

WebKit

Reporter:

Xianzhu Wang <wangxianzhu>

Component:

Layout and Rendering

Assignee:

Xianzhu Wang <wangxianzhu>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

cc-bugs, enne, jamesr, vangelis, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
patch	none

Xianzhu Wang

Reported 2012-02-09 10:26:30 PST

For now TiledLayerChromium::protectVisibleTileTextures() protects the whole layer: protectTileTextures(IntRect(IntPoint::zero(), contentBounds())); Should this be changed to protectTileTextures(visibleLayerRect())?

Attachments
patch (1.30 KB, patch) 2012-02-09 12:02 PST, Xianzhu Wang	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Adrienne Walker

Comment 1 2012-02-09 10:57:58 PST

I think you're right. I'm not sure why it wasn't this way to begin with.

Xianzhu Wang

Comment 2 2012-02-09 12:02:31 PST

Created attachment 126343 [details] patch

WebKit Review Bot

Comment 3 2012-02-09 21:03:10 PST

Comment on attachment 126343 [details] patch Clearing flags on attachment: 126343 Committed r107352: <http://trac.webkit.org/changeset/107352>

WebKit Review Bot

Comment 4 2012-02-09 21:03:13 PST

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.