Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1321)

Issue 6190067: Make Clip Cache use mask size for check for reuse (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 2 months ago by robertphillips
Modified:
12 years, 2 months ago
Reviewers:
bsalomon
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Using the render target's size doesn't make much sense for the AA clip mask cache. This patch switches over to using the size /bounds of the mask

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -58 lines) Patch
M src/gpu/GrClipMaskManager.h View 5 chunks +4 lines, -38 lines 0 comments Download
M src/gpu/GrClipMaskManager.cpp View 3 chunks +9 lines, -7 lines 0 comments Download
M tests/ClipCacheTest.cpp View 5 chunks +8 lines, -13 lines 0 comments Download

Messages

Total messages: 4
robertphillips
12 years, 2 months ago (2012-05-10 21:52:41 UTC) #1
bsalomon
On 2012/05/10 21:52:41, robertphillips wrote: LGTM
12 years, 2 months ago (2012-05-11 12:15:21 UTC) #2
robertphillips
committed as r3903
12 years, 2 months ago (2012-05-11 12:54:27 UTC) #3
robertphillips
12 years, 2 months ago (2012-05-11 13:02:12 UTC) #4
fixed compilation issue in ClipCacheTest unit test in r3904
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b