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

Issue 5376094: Remove GrGpuGLFixed (Closed)

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

Patch Set 1 #

Patch Set 2 : remove engine param from interface validate() #

Patch Set 3 : more cleanup #

Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -667 lines) Patch
M gyp/gpu.gyp View 1 chunk +0 lines, -2 lines 0 comments Download
M include/gpu/GrContext.h View 1 2 1 chunk +0 lines, -5 lines 0 comments Download
M include/gpu/GrGLConfig.h View 1 chunk +0 lines, -12 lines 0 comments Download
M include/gpu/GrGLInterface.h View 1 3 chunks +2 lines, -13 lines 0 comments Download
M src/gpu/GrContext.cpp View 1 2 1 chunk +0 lines, -4 lines 0 comments Download
M src/gpu/GrGLInterface.cpp View 1 3 chunks +41 lines, -88 lines 0 comments Download
M src/gpu/GrGLProgram.cpp View 4 chunks +4 lines, -4 lines 0 comments Download
M src/gpu/GrGLTexture.h View 1 2 1 chunk +1 line, -1 line 0 comments Download
M src/gpu/GrGLTexture.cpp View 1 2 1 chunk +3 lines, -14 lines 0 comments Download
M src/gpu/GrGpu.h View 1 chunk +1 line, -1 line 0 comments Download
M src/gpu/GrGpuFactory.cpp View 1 3 chunks +5 lines, -21 lines 0 comments Download
M src/gpu/GrGpuGL.cpp View 1 2 9 chunks +14 lines, -46 lines 0 comments Download
D src/gpu/GrGpuGLFixed.h View 1 chunk +0 lines, -66 lines 0 comments Download
D src/gpu/GrGpuGLFixed.cpp View 1 chunk +0 lines, -382 lines 0 comments Download
M src/gpu/GrGpuGLShaders.cpp View 1 chunk +1 line, -2 lines 0 comments Download
M src/gpu/app-android.cpp View 1 chunk +0 lines, -1 line 0 comments Download
M src/gpu/ios/GrGLDefaultInterface_iOS.cpp View 2 chunks +1 line, -4 lines 0 comments Download
M tests/GLInterfaceValidation.cpp View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4
bsalomon
12 years, 7 months ago (2011-11-14 18:13:58 UTC) #1
reed1
lgtm
12 years, 7 months ago (2011-11-14 18:28:16 UTC) #2
TomH
LGTM
12 years, 7 months ago (2011-11-14 19:24:37 UTC) #3
bsalomon
12 years, 7 months ago (2011-11-15 14:37:12 UTC) #4
On 2011/11/14 19:24:37, TomH wrote:
> LGTM

Closed with 2678
Sign in to reply to this message.

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