Attachment 339768 Details for Bug 183798 – Patch

[patch] Patch

bug-183798-20180507164546.patch (text/plain), 2.57 KB, created by Said Abou-Hallawa on 2018-05-07 16:45:46 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Said Abou-Hallawa

Created: 2018-05-07 16:45:46 PDT

Size: 2.57 KB

patch

obsolete

>Index: Source/WebCore/ChangeLog
>===================================================================
>--- Source/WebCore/ChangeLog	(revision 231455)
>+++ Source/WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,20 @@
>+2018-05-07  Said Abou-Hallawa  <sabouhallawa@apple.com>
>+
>+        feTurbulence is not rendered correctly on Retina display
>+        https://bugs.webkit.org/show_bug.cgi?id=183798
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        On 2x display the feTurbulence filter creates a scaled ImageBuffer but
>+        processes only the unscaled size. This is a remaining work of r168577 and
>+        is very similar to what was done for the feMorphology filter in r188271.
>+
>+        Test: fast/hidpi/filters-turbulence.html
>+
>+        * platform/graphics/filters/FETurbulence.cpp:
>+        (WebCore::FETurbulence::fillRegion const):
>+        (WebCore::FETurbulence::platformApplySoftware):
>+
> 2018-05-07  Chris Dumez  <cdumez@apple.com>
> 
>         ASSERT(!childItemWithTarget(child->target())) is hit in HistoryItem::addChildItem()
>Index: Source/WebCore/platform/graphics/filters/FETurbulence.cpp
>===================================================================
>--- Source/WebCore/platform/graphics/filters/FETurbulence.cpp	(revision 231455)
>+++ Source/WebCore/platform/graphics/filters/FETurbulence.cpp	(working copy)
>@@ -370,6 +370,7 @@ void FETurbulence::fillRegion(Uint8Clamp
>     ASSERT(endY > startY);
> 
>     IntRect filterRegion = absolutePaintRect();
>+    filterRegion.scale(filter().filterScale());
>     FloatPoint point(0, filterRegion.y() + startY);
>     int indexOfPixelChannel = startY * (filterRegion.width() << 2);
>     AffineTransform inverseTransfrom = filter().absoluteTransform().inverse().value_or(AffineTransform());
>@@ -398,7 +399,10 @@ void FETurbulence::platformApplySoftware
>     if (!pixelArray)
>         return;
> 
>-    if (absolutePaintRect().isEmpty()) {
>+    IntSize resultSize(absolutePaintRect().size());
>+    resultSize.scale(filter().filterScale());
>+
>+    if (resultSize.isEmpty()) {
>         pixelArray->zeroFill();
>         return;
>     }
>@@ -412,11 +416,11 @@ void FETurbulence::platformApplySoftware
>     PaintingData paintingData(m_seed, tileSize, baseFrequencyX, baseFrequencyY);
>     initPaint(paintingData);
> 
>-    auto area = absolutePaintRect().area();
>+    auto area = resultSize.area();
>     if (area.hasOverflowed())
>         return;
> 
>-    int height = absolutePaintRect().height();
>+    int height = resultSize.height();
> 
>     unsigned maxNumThreads = height / 8;
>     unsigned optimalThreadNumber = std::min<unsigned>(area.unsafeGet() / s_minimalRectDimension, maxNumThreads);

Actions: View | Formatted Diff | Diff

Attachments on bug 183798: 339479 | 339571 | 339573 | 339768 | 339781