Chromium Code Reviews| Index: Source/platform/graphics/skia/NativeImageSkia.cpp | 
| diff --git a/Source/platform/graphics/skia/NativeImageSkia.cpp b/Source/platform/graphics/skia/NativeImageSkia.cpp | 
| index 618669d9e6dfbd78fbadf9fe6febb6389bd18238..1952f6b951dfb05d047ef6497c1f33b34ea0c0c6 100644 | 
| --- a/Source/platform/graphics/skia/NativeImageSkia.cpp | 
| +++ b/Source/platform/graphics/skia/NativeImageSkia.cpp | 
| @@ -262,9 +262,10 @@ void NativeImageSkia::drawPattern( | 
| // Bicubic filter is only applied to defer-decoded images, see | 
| // NativeImageSkia::draw for details. | 
| bool useBicubicFilter = resampling == InterpolationHigh && isLazyDecoded; | 
| 
 
f(malita)
2014/07/25 15:20:03
I just realized that we don't need useBicubicFilte
 
 | 
| - | 
| + bool isResampled = false; | 
| 
 
f(malita)
2014/07/25 15:06:36
Instead of a new flag, I think it would be clearer
 
 | 
| if (resampling == InterpolationHigh && !useBicubicFilter) { | 
| // Do nice resampling. | 
| + isResampled = true; | 
| float scaleX = destBitmapWidth / normSrcRect.width(); | 
| float scaleY = destBitmapHeight / normSrcRect.height(); | 
| SkRect scaledSrcRect; | 
| @@ -310,7 +311,7 @@ void NativeImageSkia::drawPattern( | 
| paint.setShader(shader.get()); | 
| paint.setXfermode(WebCoreCompositeToSkiaComposite(compositeOp, blendMode).get()); | 
| paint.setColorFilter(context->colorFilter()); | 
| - paint.setFilterLevel(convertToSkiaFilterLevel(useBicubicFilter, resampling)); | 
| + paint.setFilterLevel(isResampled ? SkPaint::kNone_FilterLevel : static_cast<SkPaint::FilterLevel>(resampling)); | 
| if (isLazyDecoded) | 
| PlatformInstrumentation::didDrawLazyPixelRef(bitmap().getGenerationID()); |