Separate out the texture matrix from the coord-system change matrix in GrSamplerState. This is a step towards moving texture matrix to GrCustomStage.
R=robertphillips@google.com
Review URL: https://codereview.appspot.com/6721047
git-svn-id: http://skia.googlecode.com/svn/trunk@5973 2bbb7eff-a529-9590-31e7-b0007b416f81
diff --git a/src/gpu/GrDefaultPathRenderer.cpp b/src/gpu/GrDefaultPathRenderer.cpp
index 4f725af..2b9179f 100644
--- a/src/gpu/GrDefaultPathRenderer.cpp
+++ b/src/gpu/GrDefaultPathRenderer.cpp
@@ -450,6 +450,7 @@
drawState->disableState(GrDrawState::kNoColorWrites_StateBit);
}
GrRect bounds;
+ GrDrawState::AutoDeviceCoordDraw adcd;
if (reverse) {
GrAssert(NULL != drawState->getRenderTarget());
// draw over the whole world.
@@ -462,12 +463,7 @@
drawState->getViewInverse(&vmi)) {
vmi.mapRect(&bounds);
} else {
- const GrMatrix& vm = drawState->getViewMatrix();
- if (!drawState->preConcatSamplerMatricesWithInverse(vm)) {
- GrPrintf("Could not invert matrix.\n");
- return false;
- }
- drawState->viewMatrix()->reset();
+ adcd.set(drawState);
}
} else {
bounds = path.getBounds();
diff --git a/src/gpu/GrDrawState.cpp b/src/gpu/GrDrawState.cpp
index cc11f6c..c95049a 100644
--- a/src/gpu/GrDrawState.cpp
+++ b/src/gpu/GrDrawState.cpp
@@ -54,7 +54,7 @@
fDrawState->setViewMatrix(fViewMatrix);
for (int s = 0; s < GrDrawState::kNumStages; ++s) {
if (fRestoreMask & (1 << s)) {
- fDrawState->sampler(s)->setMatrixDeprecated(fSamplerMatrices[s]);
+ fDrawState->sampler(s)->restoreCoordChange(fSavedCoordChanges[s]);
}
}
}
@@ -77,8 +77,8 @@
for (int s = 0; s < GrDrawState::kNumStages; ++s) {
if (!(explicitCoordStageMask & (1 << s)) && drawState->isStageEnabled(s)) {
fRestoreMask |= (1 << s);
- fSamplerMatrices[s] = drawState->sampler(s)->getMatrix();
- drawState->sampler(s)->preConcatMatrix(preconcatMatrix);
+ fDrawState->sampler(s)->saveCoordChange(&fSavedCoordChanges[s]);
+ drawState->sampler(s)->preConcatCoordChange(preconcatMatrix);
}
}
}
@@ -90,7 +90,7 @@
fDrawState->setViewMatrix(fViewMatrix);
for (int s = 0; s < GrDrawState::kNumStages; ++s) {
if (fRestoreMask & (1 << s)) {
- fDrawState->sampler(s)->setMatrixDeprecated(fSamplerMatrices[s]);
+ fDrawState->sampler(s)->restoreCoordChange(fSavedCoordChanges[s]);
}
}
}
@@ -124,8 +124,8 @@
}
fRestoreMask |= (1 << s);
GrSamplerState* sampler = drawState->sampler(s);
- fSamplerMatrices[s] = sampler->getMatrix();
- sampler->preConcatMatrix(invVM);
+ sampler->saveCoordChange(&fSavedCoordChanges[s]);
+ sampler->preConcatCoordChange(invVM);
}
}
drawState->viewMatrix()->reset();
diff --git a/src/gpu/GrDrawState.h b/src/gpu/GrDrawState.h
index f3d5e37..7bfb5e5 100644
--- a/src/gpu/GrDrawState.h
+++ b/src/gpu/GrDrawState.h
@@ -10,7 +10,6 @@
#include "GrColor.h"
#include "GrMatrix.h"
-#include "GrNoncopyable.h"
#include "GrRefCnt.h"
#include "GrSamplerState.h"
#include "GrStencil.h"
@@ -267,32 +266,33 @@
}
/**
- * Preconcats the matrix of all samplers of enabled stages with a matrix.
+ * Called when the source coord system is changing. preConcat gives the transformation from the
+ * old coord system to the new coord system.
*/
- void preConcatSamplerMatrices(const GrMatrix& matrix) {
+ void preConcatSamplerMatrices(const GrMatrix& preConcat) {
for (int i = 0; i < kNumStages; ++i) {
if (this->isStageEnabled(i)) {
- fSamplerStates[i].preConcatMatrix(matrix);
+ fSamplerStates[i].preConcatCoordChange(preConcat);
}
}
}
/**
- * Preconcats the matrix of all samplers in the mask with the inverse of a
- * matrix. If the matrix inverse cannot be computed (and there is at least
- * one enabled stage) then false is returned.
+ * Called when the source coord system is changing. preConcatInverse is the inverse of the
+ * transformation from the old coord system to the new coord system. Returns false if the matrix
+ * cannot be inverted.
*/
- bool preConcatSamplerMatricesWithInverse(const GrMatrix& matrix) {
+ bool preConcatSamplerMatricesWithInverse(const GrMatrix& preConcatInverse) {
GrMatrix inv;
bool computed = false;
for (int i = 0; i < kNumStages; ++i) {
if (this->isStageEnabled(i)) {
- if (!computed && !matrix.invert(&inv)) {
+ if (!computed && !preConcatInverse.invert(&inv)) {
return false;
} else {
computed = true;
}
- fSamplerStates[i].preConcatMatrix(inv);
+ fSamplerStates[i].preConcatCoordChange(preConcatInverse);
}
}
return true;
@@ -503,10 +503,10 @@
bool isSet() const { return NULL != fDrawState; }
private:
- GrDrawState* fDrawState;
- GrMatrix fViewMatrix;
- GrMatrix fSamplerMatrices[GrDrawState::kNumStages];
- uint32_t fRestoreMask;
+ GrDrawState* fDrawState;
+ GrMatrix fViewMatrix;
+ GrSamplerState::SavedCoordChange fSavedCoordChanges[GrDrawState::kNumStages];
+ uint32_t fRestoreMask;
};
////////////////////////////////////////////////////////////////////////////
@@ -557,10 +557,10 @@
void restore();
private:
- GrDrawState* fDrawState;
- GrMatrix fViewMatrix;
- GrMatrix fSamplerMatrices[GrDrawState::kNumStages];
- uint32_t fRestoreMask;
+ GrDrawState* fDrawState;
+ GrMatrix fViewMatrix;
+ GrSamplerState::SavedCoordChange fSavedCoordChanges[GrDrawState::kNumStages];
+ uint32_t fRestoreMask;
};
/// @}
diff --git a/src/gpu/gl/GrGpuGL.h b/src/gpu/gl/GrGpuGL.h
index d2a22d0..351ff53 100644
--- a/src/gpu/gl/GrGpuGL.h
+++ b/src/gpu/gl/GrGpuGL.h
@@ -151,11 +151,9 @@
static void AdjustTextureMatrix(const GrGLTexture* texture,
GrMatrix* matrix);
- // subclass may try to take advantage of identity tex matrices.
- // This helper determines if matrix will be identity after all
- // adjustments are applied.
- static bool TextureMatrixIsIdentity(const GrGLTexture* texture,
- const GrSamplerState& sampler);
+ // This helper determines if what optimizations can be applied to the matrix after any coord
+ // adjustments are applied. The return is a bitfield of GrGLProgram::StageDesc::OptFlags.
+ static int TextureMatrixOptFlags(const GrGLTexture* texture, const GrSamplerState& sampler);
static bool BlendCoeffReferencesConstant(GrBlendCoeff coeff);
diff --git a/src/gpu/gl/GrGpuGL_program.cpp b/src/gpu/gl/GrGpuGL_program.cpp
index 9e3a5d8..e579331 100644
--- a/src/gpu/gl/GrGpuGL_program.cpp
+++ b/src/gpu/gl/GrGpuGL_program.cpp
@@ -177,19 +177,20 @@
}
}
-bool GrGpuGL::TextureMatrixIsIdentity(const GrGLTexture* texture,
- const GrSamplerState& sampler) {
+int GrGpuGL::TextureMatrixOptFlags(const GrGLTexture* texture,
+ const GrSamplerState& sampler) {
GrAssert(NULL != texture);
- if (!sampler.getMatrix().isIdentity()) {
- return false;
+ GrMatrix matrix;
+ sampler.getTotalMatrix(&matrix);
+
+ bool canBeIndentity = GrGLTexture::kTopDown_Orientation == texture->orientation();
+
+ if (canBeIndentity && matrix.isIdentity()) {
+ return GrGLProgram::StageDesc::kIdentityMatrix_OptFlagBit;
+ } else if (!matrix.hasPerspective()) {
+ return GrGLProgram::StageDesc::kNoPerspective_OptFlagBit;
}
- GrGLTexture::Orientation orientation = texture->orientation();
- if (GrGLTexture::kBottomUp_Orientation == orientation) {
- return false;
- } else {
- GrAssert(GrGLTexture::kTopDown_Orientation == orientation);
- }
- return true;
+ return 0;
}
///////////////////////////////////////////////////////////////////////////////
@@ -208,7 +209,8 @@
UniformHandle matrixUni = fCurrentProgram->fUniforms.fStages[s].fTextureMatrixUni;
const GrMatrix& hwMatrix = fCurrentProgram->fTextureMatrices[s];
- const GrMatrix& samplerMatrix = drawState.getSampler(s).getMatrix();
+ GrMatrix samplerMatrix;
+ drawState.getSampler(s).getTotalMatrix(&samplerMatrix);
if (kInvalidUniformHandle != matrixUni &&
(orientationChange || !hwMatrix.cheapEqualTo(samplerMatrix))) {
@@ -238,7 +240,6 @@
}
}
-
void GrGpuGL::flushColorMatrix() {
UniformHandle matrixUni = fCurrentProgram->fUniforms.fColorMatrixUni;
UniformHandle vecUni = fCurrentProgram->fUniforms.fColorMatrixVecUni;
@@ -701,15 +702,13 @@
// FIXME: Still assuming one texture per custom stage
const GrCustomStage* customStage = drawState.getSampler(s).getCustomStage();
const GrGLTexture* texture = static_cast<const GrGLTexture*>(customStage->texture(0));
+ GrMatrix samplerMatrix;
+ sampler.getTotalMatrix(&samplerMatrix);
if (NULL != texture) {
// We call this helper function rather then simply checking the client-specified
// texture matrix. This is because we may have to concat a y-inversion to account
// for texture orientation.
- if (TextureMatrixIsIdentity(texture, sampler)) {
- stage.fOptFlags |= StageDesc::kIdentityMatrix_OptFlagBit;
- } else if (!sampler.getMatrix().hasPerspective()) {
- stage.fOptFlags |= StageDesc::kNoPerspective_OptFlagBit;
- }
+ stage.fOptFlags |= TextureMatrixOptFlags(texture, sampler);
}
setup_custom_stage(&stage, sampler, this->glCaps(), customStages,