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

Issue 6506086: Consolidate texture access functions, provide default GrTextureAccess (Closed)

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

Patch Set 1 #

Patch Set 2 : update #

Patch Set 3 : remove TextureSampler::fTexture #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+49 lines, -110 lines) Patch
M src/gpu/effects/GrColorTableEffect.cpp View 1 chunk +8 lines, -8 lines 0 comments Download
M src/gpu/effects/GrConfigConversionEffect.cpp View 1 1 chunk +0 lines, -1 line 0 comments Download
M src/gpu/gl/GrGLProgram.h View 1 chunk +4 lines, -12 lines 1 comment Download
M src/gpu/gl/GrGLProgram.cpp View 2 chunks +23 lines, -6 lines 0 comments Download
M src/gpu/gl/GrGLShaderBuilder.h View 1 2 7 chunks +8 lines, -47 lines 0 comments Download
M src/gpu/gl/GrGLShaderBuilder.cpp View 3 chunks +5 lines, -28 lines 0 comments Download
M src/gpu/gl/GrGpuGL_program.cpp View 1 chunk +1 line, -7 lines 0 comments Download
M tests/GLProgramsTest.cpp View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 3
bsalomon
The next step after this CL is to make all the GrCustomStage subclasses provide their ...
12 years, 3 months ago (2012-09-06 22:00:13 UTC) #1
robertphillips
LGTM
12 years, 3 months ago (2012-09-06 22:12:39 UTC) #2
bsalomon
12 years, 3 months ago (2012-09-07 13:54:35 UTC) #3
Closed with r5428
Sign in to reply to this message.

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