Attachment 371938 Details for Bug 198746 – WIP

[patch] WIP

wk-clip-to-image.diff (text/plain), 2.98 KB, created by Carlos Garcia Campos on 2019-06-12 02:25:29 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Carlos Garcia Campos

Created: 2019-06-12 02:25:29 PDT

Size: 2.98 KB

patch

obsolete

>diff --git a/Source/WebCore/platform/graphics/cairo/PlatformContextCairo.cpp b/Source/WebCore/platform/graphics/cairo/PlatformContextCairo.cpp
>index 7f242fdbecb..b37d184b59c 100644
>--- a/Source/WebCore/platform/graphics/cairo/PlatformContextCairo.cpp
>+++ b/Source/WebCore/platform/graphics/cairo/PlatformContextCairo.cpp
>@@ -30,41 +30,17 @@
> 
> #if USE(CAIRO)
> 
>-#include "CairoUtilities.h"
>-#include "Gradient.h"
>-#include "GraphicsContext.h"
>-#include "Pattern.h"
> #include <cairo.h>
> 
> namespace WebCore {
> 
>-// In Cairo image masking is immediate, so to emulate image clipping we must save masking
>-// details as part of the context state and apply them during platform restore.
>-class ImageMaskInformation {
>-public:
>-    void update(cairo_surface_t* maskSurface, const FloatRect& maskRect)
>-    {
>-        m_maskSurface = maskSurface;
>-        m_maskRect = maskRect;
>-    }
>-
>-    bool isValid() const { return m_maskSurface; }
>-    cairo_surface_t* maskSurface() const { return m_maskSurface.get(); }
>-    const FloatRect& maskRect() const { return m_maskRect; }
>-
>-private:
>-    RefPtr<cairo_surface_t> m_maskSurface;
>-    FloatRect m_maskRect;
>-};
>-
>-
> // Encapsulates the additional painting state information we store for each
> // pushed graphics state.
> class PlatformContextCairo::State {
> public:
>     State() = default;
> 
>-    ImageMaskInformation m_imageMaskInformation;
>+    RefPtr<cairo_pattern_t> m_maskPattern;
> };
> 
> PlatformContextCairo::PlatformContextCairo(cairo_t* cr)
>@@ -76,11 +52,9 @@ PlatformContextCairo::PlatformContextCairo(cairo_t* cr)
> 
> void PlatformContextCairo::restore()
> {
>-    const ImageMaskInformation& maskInformation = m_state->m_imageMaskInformation;
>-    if (maskInformation.isValid()) {
>-        const FloatRect& maskRect = maskInformation.maskRect();
>+    if (m_state->m_maskPattern) {
>         cairo_pop_group_to_source(m_cr.get());
>-        cairo_mask_surface(m_cr.get(), maskInformation.maskSurface(), maskRect.x(), maskRect.y());
>+        cairo_mask(m_cr.get(), m_state->m_maskPattern.get());
>     }
> 
>     m_stateStack.removeLast();
>@@ -105,11 +79,14 @@ void PlatformContextCairo::pushImageMask(cairo_surface_t* surface, const FloatRe
>     // We must call savePlatformState at least once before we can use image masking,
>     // since we actually apply the mask in restorePlatformState.
>     ASSERT(!m_stateStack.isEmpty());
>-    m_state->m_imageMaskInformation.update(surface, rect);
>+    m_state->m_maskPattern = adoptRef(cairo_pattern_create_for_surface(surface));
>+
>+    cairo_matrix_t matrix;
>+    cairo_matrix_init_translate(&matrix, rect.x(), rect.y());
>+    cairo_matrix_invert(&matrix);
>+    cairo_pattern_set_matrix(m_state->m_maskPattern.get(), &matrix);
>+    cairo_pattern_set_extend(m_state->m_maskPattern.get(), CAIRO_EXTEND_PAD);
> 
>-    // Cairo doesn't support the notion of an image clip, so we push a group here
>-    // and then paint it to the surface with an image mask (which is an immediate
>-    // operation) during restorePlatformState.
>     cairo_push_group(m_cr.get());
> }
>

Actions: View | Formatted Diff | Diff

Attachments on bug 198746: 371826 | 371938 | 371942 | 371944 | 371946 | 372014 | 372015