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

Issue 6125046: Refactoring to begin encapsulating SW path rendering in GrPathRenderer (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

First pass at moving the software path rendering into a GrPathRenderer-derived class. I currently left the software rendering path in GrContext.cpp but will pull it into the GrSoftwarePathRenderer shell in a follow on delivery.

Patch Set 1 #

Patch Set 2 : Removed dead code & added comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+136 lines, -39 lines) Patch
M gyp/gpu.gyp View 1 chunk +2 lines, -0 lines 0 comments Download
M src/gpu/GrAddPathRenderers_default.cpp View 2 chunks +2 lines, -0 lines 0 comments Download
M src/gpu/GrContext.cpp View 5 chunks +59 lines, -39 lines 0 comments Download
A src/gpu/GrSoftwarePathRenderer.h View 1 chunk +38 lines, -0 lines 0 comments Download
A src/gpu/GrSoftwarePathRenderer.cpp View 1 1 chunk +35 lines, -0 lines 0 comments Download

Messages

Total messages: 4
robertphillips
12 years, 2 months ago (2012-04-26 00:05:42 UTC) #1
bsalomon
On 2012/04/26 00:05:42, robertphillips wrote: LGTM
12 years, 2 months ago (2012-04-26 13:32:26 UTC) #2
robertphillips
Committed first portion as r3769 Holding off on gpu.gyp & GrAddPathRenderers_default.cpp until can do the ...
12 years, 2 months ago (2012-04-27 12:10:13 UTC) #3
robertphillips
12 years, 2 months ago (2012-05-01 12:21:12 UTC) #4
Committed second half as r3804
Sign in to reply to this message.

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