Remove unneeded field from GrGLTexture

git-svn-id: http://skia.googlecode.com/svn/trunk@2607 2bbb7eff-a529-9590-31e7-b0007b416f81
This commit is contained in:
bsalomon@google.com 2011-11-05 13:25:21 +00:00
parent b635d39172
commit 9d6cfd80a0
3 changed files with 32 additions and 34 deletions

View File

@ -47,7 +47,6 @@ void GrGLTexture::init(GrGpuGL* gpu,
textureDesc.fTextureID,
textureDesc.fOwnsID);
fUploadFormat = textureDesc.fUploadFormat;
fUploadByteCount = textureDesc.fUploadByteCount;
fUploadType = textureDesc.fUploadType;
fOrientation = textureDesc.fOrientation;
fScaleX = GrIntToScalar(textureDesc.fContentWidth) /
@ -114,7 +113,11 @@ void GrGLTexture::uploadTextureData(int x,
int height,
const void* srcData,
size_t rowBytes) {
GrIRect bounds = GrIRect::MakeWH(this->width(), this->height());
GrIRect subrect = GrIRect::MakeXYWH(x,y,width, height);
if (!bounds.contains(subrect)) {
return;
}
GPUGL->setSpareTextureUnit();
// ES2 glCompressedTexSubImage2D doesn't support any formats
@ -124,8 +127,10 @@ void GrGLTexture::uploadTextureData(int x,
// in case we need a temporary, trimmed copy of the src pixels
SkAutoSMalloc<128 * 128> tempStorage;
size_t bpp = GrBytesPerPixel(this->config());
size_t trimRowBytes = width * bpp;
if (!rowBytes) {
rowBytes = fUploadByteCount * width;
rowBytes = trimRowBytes;
}
/*
* check whether to allocate a temporary buffer for flipping y or
@ -137,14 +142,13 @@ void GrGLTexture::uploadTextureData(int x,
bool flipY = kBottomUp_Orientation == fOrientation;
if (kDesktop_GrGLBinding == GPUGL->glBinding() && !flipY) {
// can't use this for flipping, only non-neg values allowed. :(
if (srcData && rowBytes) {
GL_CALL(PixelStorei(GR_GL_UNPACK_ROW_LENGTH,
rowBytes / fUploadByteCount));
if (srcData && rowBytes != trimRowBytes) {
GrGLint rowLength = static_cast<GrGLint>(rowBytes / bpp);
GL_CALL(PixelStorei(GR_GL_UNPACK_ROW_LENGTH, rowLength));
restoreGLRowLength = true;
}
} else {
size_t trimRowBytes = width * fUploadByteCount;
if (srcData && (trimRowBytes < rowBytes || flipY)) {
if (srcData && (trimRowBytes != rowBytes || flipY)) {
// copy the data into our new storage, skipping the trailing bytes
size_t trimSize = height * trimRowBytes;
const char* src = (const char*)srcData;
@ -170,7 +174,7 @@ void GrGLTexture::uploadTextureData(int x,
y = this->height() - (y + height);
}
GL_CALL(BindTexture(GR_GL_TEXTURE_2D, fTexIDObj->id()));
GL_CALL(PixelStorei(GR_GL_UNPACK_ALIGNMENT, fUploadByteCount));
GL_CALL(PixelStorei(GR_GL_UNPACK_ALIGNMENT, static_cast<GrGLint>(bpp)));
GL_CALL(TexSubImage2D(GR_GL_TEXTURE_2D, 0, x, y, width, height,
fUploadFormat, fUploadType, srcData));

View File

@ -101,7 +101,6 @@ public:
GrGLuint textureID() const { return fTexIDObj->id(); }
GrGLenum uploadFormat() const { return fUploadFormat; }
GrGLenum uploadByteCount() const { return fUploadByteCount; }
GrGLenum uploadType() const { return fUploadType; }
/**
@ -136,7 +135,6 @@ private:
TexParams fTexParams;
GrGLTexID* fTexIDObj;
GrGLenum fUploadFormat;
GrGLenum fUploadByteCount;
GrGLenum fUploadType;
// precomputed content / alloc ratios
GrScalar fScaleX;

View File

@ -690,7 +690,6 @@ GrResource* GrGpuGL::onCreatePlatformSurface(const GrPlatformSurfaceDesc& desc)
texDesc.fFormat = desc.fConfig;
texDesc.fOrientation = GrGLTexture::kBottomUp_Orientation;
texDesc.fTextureID = desc.fPlatformTexture;
texDesc.fUploadByteCount = GrBytesPerPixel(desc.fConfig);
texDesc.fOwnsID = false;
params.invalidate(); // rather than do glGets.
@ -721,9 +720,13 @@ void GrGpuGL::allocateAndUploadTexData(const GrGLTexture::Desc& desc,
GrGLenum internalFormat,
const void* data,
size_t rowBytes) {
// we assume the texture is bound;
// we assume the texture is bound
size_t bpp = GrBytesPerPixel(desc.fFormat);
size_t trimRowBytes = desc.fContentWidth * desc.fUploadByteCount;
if (!rowBytes) {
rowBytes = desc.fUploadByteCount * desc.fContentWidth;
rowBytes = trimRowBytes;
}
// in case we need a temporary, trimmed copy of the src pixels
@ -737,13 +740,12 @@ void GrGpuGL::allocateAndUploadTexData(const GrGLTexture::Desc& desc,
*/
bool flipY = GrGLTexture::kBottomUp_Orientation == desc.fOrientation;
if (kDesktop_GrGLBinding == this->glBinding() && !flipY) {
if (data && rowBytes != desc.fContentWidth * desc.fUploadByteCount) {
GL_CALL(PixelStorei(GR_GL_UNPACK_ROW_LENGTH,
rowBytes / desc.fUploadByteCount));
if (data && rowBytes != trimRowBytes) {
GrGLint rowLength = static_cast<GrGLint>(rowBytes / bpp);
GL_CALL(PixelStorei(GR_GL_UNPACK_ROW_LENGTH, rowLength));
}
} else {
size_t trimRowBytes = desc.fContentWidth * desc.fUploadByteCount;
if (data && (trimRowBytes < rowBytes || flipY)) {
if (data && (trimRowBytes != rowBytes || flipY)) {
// copy the data into our new storage, skipping the trailing bytes
size_t trimSize = desc.fContentHeight * trimRowBytes;
const char* src = (const char*)data;
@ -766,7 +768,7 @@ void GrGpuGL::allocateAndUploadTexData(const GrGLTexture::Desc& desc,
}
}
GL_CALL(PixelStorei(GR_GL_UNPACK_ALIGNMENT, desc.fUploadByteCount));
GL_CALL(PixelStorei(GR_GL_UNPACK_ALIGNMENT, static_cast<GrGLint>(bpp)));
if (kIndex_8_GrPixelConfig == desc.fFormat &&
this->getCaps().f8BitPaletteSupport) {
// ES only supports CompressedTexImage2D, not CompressedTexSubimage2D
@ -795,20 +797,16 @@ void GrGpuGL::allocateAndUploadTexData(const GrGLTexture::Desc& desc,
maxTexels = GrMax(extraW * desc.fContentHeight, maxTexels);
maxTexels = GrMax(desc.fContentWidth * extraH, maxTexels);
SkAutoSMalloc<128*128> texels(desc.fUploadByteCount * maxTexels);
SkAutoSMalloc<128*128> texels(bpp * maxTexels);
// rowBytes is actual stride between rows in data
// rowDataBytes is the actual amount of non-pad data in a row
// and the stride used for uploading extraH rows.
uint32_t rowDataBytes = desc.fContentWidth * desc.fUploadByteCount;
if (extraH) {
uint8_t* lastRowStart = (uint8_t*) data +
(desc.fContentHeight - 1) * rowBytes;
uint8_t* extraRowStart = (uint8_t*)texels.get();
for (int i = 0; i < extraH; ++i) {
memcpy(extraRowStart, lastRowStart, rowDataBytes);
extraRowStart += rowDataBytes;
memcpy(extraRowStart, lastRowStart, trimRowBytes);
extraRowStart += trimRowBytes;
}
GL_CALL(TexSubImage2D(GR_GL_TEXTURE_2D, 0, 0,
desc.fContentHeight, desc.fContentWidth,
@ -816,13 +814,12 @@ void GrGpuGL::allocateAndUploadTexData(const GrGLTexture::Desc& desc,
desc.fUploadType, texels.get()));
}
if (extraW) {
uint8_t* edgeTexel = (uint8_t*)data +
rowDataBytes - desc.fUploadByteCount;
uint8_t* edgeTexel = (uint8_t*)data + trimRowBytes - bpp;
uint8_t* extraTexel = (uint8_t*)texels.get();
for (int j = 0; j < desc.fContentHeight; ++j) {
for (int i = 0; i < extraW; ++i) {
memcpy(extraTexel, edgeTexel, desc.fUploadByteCount);
extraTexel += desc.fUploadByteCount;
memcpy(extraTexel, edgeTexel, bpp);
extraTexel += bpp;
}
edgeTexel += rowBytes;
}
@ -833,12 +830,11 @@ void GrGpuGL::allocateAndUploadTexData(const GrGLTexture::Desc& desc,
}
if (extraW && extraH) {
uint8_t* cornerTexel = (uint8_t*)data +
desc.fContentHeight * rowBytes -
desc.fUploadByteCount;
desc.fContentHeight * rowBytes - bpp;
uint8_t* extraTexel = (uint8_t*)texels.get();
for (int i = 0; i < extraW*extraH; ++i) {
memcpy(extraTexel, cornerTexel, desc.fUploadByteCount);
extraTexel += desc.fUploadByteCount;
extraTexel += bpp;
}
GL_CALL(TexSubImage2D(GR_GL_TEXTURE_2D, 0, desc.fContentWidth,
desc.fContentHeight, extraW, extraH,