| 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;
 | 
|      }
 | 
| 
 |