Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(359)

Issue 1785873003: Implement GL support for buffer textures (Closed)

Created:
4 years, 9 months ago by Chris Dalton
Modified:
4 years, 8 months ago
Reviewers:
joshualitt, bsalomon
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@upload_glTexBuffer
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Implement GL support for buffer textures Adds a new sampler type for buffer textures and implements GL support for it. The addition of buffer textures unfortunately complicates textures now because GrTextureParams is not always applicable anymore and textures can be 1-dimensional. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1785873003

Patch Set 1 #

Patch Set 2 : rebse #

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+148 lines, -54 lines) Patch
M include/gpu/GrTexture.h View 1 chunk +6 lines, -0 lines 0 comments Download
M include/gpu/GrTextureAccess.h View 3 chunks +10 lines, -1 line 0 comments Download
M include/gpu/GrTypesPriv.h View 7 chunks +24 lines, -17 lines 0 comments Download
M src/gpu/GrTextureAccess.cpp View 2 chunks +16 lines, -0 lines 0 comments Download
M src/gpu/gl/GrGLGpu.h View 1 2 3 4 2 chunks +6 lines, -3 lines 0 comments Download
M src/gpu/gl/GrGLGpu.cpp View 1 2 3 4 8 chunks +50 lines, -22 lines 0 comments Download
M src/gpu/gl/GrGLProgramDataManager.cpp View 1 chunk +1 line, -2 lines 0 comments Download
M src/gpu/gl/GrGLProgramDesc.cpp View 1 chunk +2 lines, -0 lines 0 comments Download
M src/gpu/gl/GrGLTexture.cpp View 1 chunk +2 lines, -0 lines 0 comments Download
M src/gpu/glsl/GrGLSL.h View 1 2 2 chunks +8 lines, -2 lines 0 comments Download
M src/gpu/glsl/GrGLSLProgramBuilder.cpp View 1 2 3 4 1 chunk +6 lines, -0 lines 0 comments Download
M src/gpu/glsl/GrGLSLShaderBuilder.h View 1 2 3 2 chunks +12 lines, -0 lines 0 comments Download
M src/gpu/glsl/GrGLSLShaderBuilder.cpp View 1 2 2 chunks +5 lines, -7 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 7 (2 generated)
Chris Dalton
It's unfortunate that buffer textures add so much complexity. Textures used to be pretty well ...
4 years, 9 months ago (2016-03-11 08:54:46 UTC) #3
bsalomon
What do you think about creating a new class for buffer textures? Under the hood ...
4 years, 9 months ago (2016-03-16 15:18:34 UTC) #4
Chris Dalton
On 2016/03/16 15:18:34, bsalomon wrote: > What do you think about creating a new class ...
4 years, 9 months ago (2016-03-16 15:25:21 UTC) #5
Chris Dalton
On 2016/03/16 15:18:34, bsalomon wrote: > What do you think about creating a new class ...
4 years, 9 months ago (2016-03-16 15:25:22 UTC) #6
bsalomon
4 years, 9 months ago (2016-03-16 15:30:57 UTC) #7
On 2016/03/16 15:25:22, Chris Dalton wrote:
> On 2016/03/16 15:18:34, bsalomon wrote:
> > What do you think about creating a new class for buffer textures? Under the
> hood
> > they use some of the same GL method (glGenTextures) but they feel
> fundamentally
> > different than what a GrTexture is.
> 
> I think that would make more sense (probably something that needs a discussion
> bigger than can be had on the code review site). One gotcha is that
> GrTextureAccess would need to be able to reference a regular texture or a
buffer
> texture, and sometimes it would have GrTextureParams and other times not.

Or should we have a base class that generalizes GrTextureAccess? If you want to
setup a VC or what not that works for me. I can pull in others from here.

Powered by Google App Engine
This is Rietveld 408576698