[poc] curve-filter

third_party/WebKit/Source/platform/graphics/Image.cpp

 /*
  * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2004, 2005, 2006 Apple Computer, Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 19 matching lines @@
 #include "platform/Length.h"
 #include "platform/MIMETypeRegistry.h"
 #include "platform/PlatformInstrumentation.h"
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/SharedBuffer.h"
 #include "platform/TraceEvent.h"
 #include "platform/geometry/FloatPoint.h"
 #include "platform/geometry/FloatRect.h"
 #include "platform/geometry/FloatSize.h"
 #include "platform/graphics/BitmapImage.h"
+#include "platform/graphics/ColorSpaceFilter.h"
+#include "platform/graphics/ColorSpaceProfile.h"
 #include "platform/graphics/DeferredImageDecoder.h"
 #include "platform/graphics/GraphicsContext.h"
+#include "platform/graphics/GraphicsScreen.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebData.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/core/SkImage.h"
 #include "third_party/skia/include/core/SkPictureRecorder.h"
 #include "wtf/MainThread.h"
 #include "wtf/StdLibExtras.h"
 
 #include <math.h>
 
@@ skipping 125 matching lines @@
         ctxt->setImageInterpolationQuality(InterpolationLow);
         drawPattern(ctxt, srcRect, tileScaleFactor, patternPhase, op, dstRect);
         ctxt->setImageInterpolationQuality(previousInterpolationQuality);
     } else {
         drawPattern(ctxt, srcRect, tileScaleFactor, patternPhase, op, dstRect);
     }
 
     startAnimation();
 }
 
-namespace {
-
-PassRefPtr<SkShader> createPatternShader(const SkImage* image, const SkMatrix& shaderMatrix,
-    const SkPaint& paint, const FloatSize& spacing)
+static PassRefPtr<SkShader> createPatternShader(const SkImage* image, SkPaint* paint, const SkMatrix& shaderMatrix, const FloatSize& spacing)
 {
     if (spacing.isZero())
         return adoptRef(image->newShader(SkShader::kRepeat_TileMode, SkShader::kRepeat_TileMode, &shaderMatrix));
 
-    // Arbitrary tiling is currently only supported for SkPictureShader - so we use it instead
-    // of a plain bitmap shader to implement spacing.
-    const SkRect tileRect = SkRect::MakeWH(
-        image->width() + spacing.width(),
-        image->height() + spacing.height());
+    // Arbitrary tiling is currently only supported for SkPictureShader so use it, instead
+    // of a plain bitmap shader, to implement spacing (CSS background-repeat: space).
+    SkRect tileRect = SkRect::MakeWH(image->width() + spacing.width(), image->height() + spacing.height());
 
     SkPictureRecorder recorder;
     SkCanvas* canvas = recorder.beginRecording(tileRect);
-    canvas->drawImage(image, 0, 0, &paint);
+    canvas->drawImage(image, 0, 0, paint);
     RefPtr<const SkPicture> picture = adoptRef(recorder.endRecordingAsPicture());
 
-    return adoptRef(SkShader::CreatePictureShader(
-        picture.get(), SkShader::kRepeat_TileMode, SkShader::kRepeat_TileMode, &shaderMatrix, nullptr));
+    // Remove any color filter from callers paint: it is applied when drawing the SkPicture
+    // shader so clearing it here prevents caller from applying the filter twice.
+    if (paint->getColorFilter())
+        paint->setColorFilter(nullptr);
+
+    return adoptRef(SkShader::CreatePictureShader(picture.get(), SkShader::kRepeat_TileMode, SkShader::kRepeat_TileMode, &shaderMatrix, nullptr));
 }
 
-} // anonymous namespace
+inline PassRefPtr<ColorSpaceProfile> sourceColorProfile(Image* image)
+{
+    return image->isBitmapImage() ? toBitmapImage(image)->colorProfile() : nullptr;
+}
 
 void Image::drawPattern(GraphicsContext* context, const FloatRect& floatSrcRect, const FloatSize& scale,
     const FloatPoint& phase, SkXfermode::Mode compositeOp, const FloatRect& destRect, const IntSize& repeatSpacing)
 {
     TRACE_EVENT0("skia", "Image::drawPattern");
 
     RefPtr<SkImage> image = imageForCurrentFrame();
     if (!image)
         return;
 
@@ skipping 16 matching lines @@
     // set to the pattern's transform, which just includes scale.
     localMatrix.preScale(scale.width(), scale.height());
 
     // Fetch this now as subsetting may swap the image.
     auto imageID = image->uniqueID();
 
     image = adoptRef(image->newSubset(enclosingIntRect(normSrcRect)));
     if (!image)
         return;
 
-    {
-        SkPaint paint = context->fillPaint();
-        paint.setColor(SK_ColorBLACK);
-        paint.setXfermodeMode(compositeOp);
-        paint.setFilterQuality(context->computeFilterQuality(this, destRect, normSrcRect));
-        paint.setAntiAlias(context->shouldAntialias());
-        RefPtr<SkShader> shader = createPatternShader(image.get(), localMatrix, paint,
-            FloatSize(repeatSpacing.width() / scale.width(), repeatSpacing.height() / scale.height()));
-        paint.setShader(shader.get());
-        context->drawRect(destRect, paint);
+    RefPtr<SkColorFilter> colorTransform;
+    if (RefPtr<ColorSpaceProfile> source = sourceColorProfile(this)) {
+        // Tagged images (those that have a color profile) are drawn color-correct on supported platforms.
+        RefPtr<ColorSpaceProfile> screen = screenColorProfile(currentScreenId());
+        if (!context->colorFilter()) // http://crbug.com/542498
+            colorTransform = createColorSpaceFilter(source.get(), screen.get());
+        RELEASE_ASSERT(currentScreenId());
+    } else {
+        // For sRGB assumed images: easy, just interpolate the sRGB color profile here as the src profile?
+        // FIXME: untagged images should be drawn from sRGB color space, per CSS2.1 onwards, a change that
+        // would also require that all CSS content, including <canvas>, be drawn from sRGB space to match,
+        // and the same for plugins (Flash, NaCl) which don't define their color space at all, <video> and
+        // OOPIF, and so on ...
     }
 
+    SkPaint paint = context->fillPaint();
+    paint.setColor(SK_ColorBLACK);
+    paint.setXfermodeMode(compositeOp);
+    paint.setFilterQuality(context->computeFilterQuality(this, destRect, normSrcRect));
+    paint.setAntiAlias(context->shouldAntialias());
+
+    // BoxPainter background paint can set a luminance color filter on the context. Add it to the paint if
+    // present: see layout test css3/masking/mask-luminance-png.html and http://crbug.com/542498
+    if (context->colorFilter()) {
+        fprintf(stderr, "Image::drawPattern color filter\n");
+        paint.setColorFilter(context->colorFilter());
+        fflush(stderr);
+    } else if (colorTransform.get()) {
+        fprintf(stderr, "Image::drawPattern color transform key [%s]\n", colorSpaceFilterKey(colorTransform.get()).latin1().data());
+        paint.setColorFilter(colorTransform.get());
+        fflush(stderr);
+    }
+
+    FloatSize spacing = FloatSize(repeatSpacing.width() / scale.width(), repeatSpacing.height() / scale.height());
+    RefPtr<SkShader> shader = createPatternShader(image.get(), &paint, localMatrix, spacing);
+    paint.setShader(shader.get());
+
+    context->drawRect(destRect, paint);
+
     if (currentFrameIsLazyDecoded())
         PlatformInstrumentation::didDrawLazyPixelRef(imageID);
 }
 
 void Image::computeIntrinsicDimensions(Length& intrinsicWidth, Length& intrinsicHeight, FloatSize& intrinsicRatio)
 {
     intrinsicRatio = FloatSize(size());
     intrinsicWidth = Length(intrinsicRatio.width(), Fixed);
     intrinsicHeight = Length(intrinsicRatio.height(), Fixed);
 }
 
 PassRefPtr<Image> Image::imageForDefaultFrame()
 {
     RefPtr<Image> image(this);
 
     return image.release();
 }
 
 bool Image::isTextureBacked()
 {
     return imageForCurrentFrame()->isTextureBacked();
 }
 
 } // namespace blink