| Index: src/gpu/GrContext.cpp
|
| diff --git a/src/gpu/GrContext.cpp b/src/gpu/GrContext.cpp
|
| index 065d61bb954a5d937712d6b8bf68b6aee86356d1..c9fb9a12ec8c716216655d27241dafb7fa4ba194 100644
|
| --- a/src/gpu/GrContext.cpp
|
| +++ b/src/gpu/GrContext.cpp
|
| @@ -370,8 +370,8 @@ bool GrContext::writeSurfacePixels(GrSurface* surface,
|
| textureMatrix.setIDiv(tempTexture->width(), tempTexture->height());
|
| GrPaint paint;
|
| if (applyPremulToSrc) {
|
| - fp.reset(this->createUPMToPMEffect(paint.getProcessorDataManager(), tempTexture,
|
| - tempDrawInfo.fSwapRAndB, textureMatrix));
|
| + fp.reset(this->createUPMToPMEffect(tempTexture, tempDrawInfo.fSwapRAndB,
|
| + textureMatrix));
|
| // If premultiplying was the only reason for the draw, fall back to a straight write.
|
| if (!fp) {
|
| if (GrGpu::kCallerPrefersDraw_DrawPreference == drawPreference) {
|
| @@ -383,8 +383,7 @@ bool GrContext::writeSurfacePixels(GrSurface* surface,
|
| }
|
| if (tempTexture) {
|
| if (!fp) {
|
| - fp.reset(GrConfigConversionEffect::Create(
|
| - paint.getProcessorDataManager(), tempTexture, tempDrawInfo.fSwapRAndB,
|
| + fp.reset(GrConfigConversionEffect::Create(tempTexture, tempDrawInfo.fSwapRAndB,
|
| GrConfigConversionEffect::kNone_PMConversion, textureMatrix));
|
| if (!fp) {
|
| return false;
|
| @@ -508,8 +507,7 @@ bool GrContext::readSurfacePixels(GrSurface* src,
|
| GrPaint paint;
|
| SkAutoTUnref<const GrFragmentProcessor> fp;
|
| if (unpremul) {
|
| - fp.reset(this->createPMToUPMEffect(
|
| - paint.getProcessorDataManager(), src->asTexture(), tempDrawInfo.fSwapRAndB,
|
| + fp.reset(this->createPMToUPMEffect(src->asTexture(), tempDrawInfo.fSwapRAndB,
|
| textureMatrix));
|
| if (fp) {
|
| unpremul = false; // we no longer need to do this on CPU after the read back.
|
| @@ -520,8 +518,7 @@ bool GrContext::readSurfacePixels(GrSurface* src,
|
| }
|
| }
|
| if (!fp && temp) {
|
| - fp.reset(GrConfigConversionEffect::Create(
|
| - paint.getProcessorDataManager(), src->asTexture(), tempDrawInfo.fSwapRAndB,
|
| + fp.reset(GrConfigConversionEffect::Create(src->asTexture(), tempDrawInfo.fSwapRAndB,
|
| GrConfigConversionEffect::kNone_PMConversion, textureMatrix));
|
| }
|
| if (fp) {
|
| @@ -697,8 +694,7 @@ void GrContext::testPMConversionsIfNecessary(uint32_t flags) {
|
| }
|
| }
|
|
|
| -const GrFragmentProcessor* GrContext::createPMToUPMEffect(GrProcessorDataManager* procDataManager,
|
| - GrTexture* texture,
|
| +const GrFragmentProcessor* GrContext::createPMToUPMEffect(GrTexture* texture,
|
| bool swapRAndB,
|
| const SkMatrix& matrix) const {
|
| // We should have already called this->testPMConversionsIfNecessary().
|
| @@ -706,15 +702,13 @@ const GrFragmentProcessor* GrContext::createPMToUPMEffect(GrProcessorDataManager
|
| GrConfigConversionEffect::PMConversion pmToUPM =
|
| static_cast<GrConfigConversionEffect::PMConversion>(fPMToUPMConversion);
|
| if (GrConfigConversionEffect::kNone_PMConversion != pmToUPM) {
|
| - return GrConfigConversionEffect::Create(procDataManager, texture, swapRAndB, pmToUPM,
|
| - matrix);
|
| + return GrConfigConversionEffect::Create(texture, swapRAndB, pmToUPM, matrix);
|
| } else {
|
| return nullptr;
|
| }
|
| }
|
|
|
| -const GrFragmentProcessor* GrContext::createUPMToPMEffect(GrProcessorDataManager* procDataManager,
|
| - GrTexture* texture,
|
| +const GrFragmentProcessor* GrContext::createUPMToPMEffect(GrTexture* texture,
|
| bool swapRAndB,
|
| const SkMatrix& matrix) const {
|
| // We should have already called this->testPMConversionsIfNecessary().
|
| @@ -722,8 +716,7 @@ const GrFragmentProcessor* GrContext::createUPMToPMEffect(GrProcessorDataManager
|
| GrConfigConversionEffect::PMConversion upmToPM =
|
| static_cast<GrConfigConversionEffect::PMConversion>(fUPMToPMConversion);
|
| if (GrConfigConversionEffect::kNone_PMConversion != upmToPM) {
|
| - return GrConfigConversionEffect::Create(procDataManager, texture, swapRAndB, upmToPM,
|
| - matrix);
|
| + return GrConfigConversionEffect::Create(texture, swapRAndB, upmToPM, matrix);
|
| } else {
|
| return nullptr;
|
| }
|
|
|