commit 77c26dae7c984f5c60206091f87897079c8a5d93
author Patrick Luby <guibmacdev@gmail.com> Fri Apr 26 20:25:03 2024 -0400
committer Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> Mon Apr 29 20:21:50 2024 +0200
tree dcabaa1d574f20a10a9601cd8b6ce83a8537f3d5
parent 0d37975ae718e1baa312616dfbbd71496e077a49

tdf#153306 prevent subpixel shifting of X coordinate

HACK: for some unknown reason, if the X coordinate of the
path's bounds is more than 1024, SkBlendMode::kExclusion will
shift by about a half a pixel to the right with Skia/Metal on
a Retina display. Weirdly, if the same polygon is repeatedly
drawn, the total shift is cumulative so if the drawn polygon
is more than a few pixels wide, the blinking cursor in Writer
will exhibit this bug but only for one thin vertical slice at
a time. Apparently, shifting drawing a very tiny amount to
the left seems to be enough to quell this runaway cumulative
X coordinate shift.

Change-Id: Ic1ac8a390df51c4aa1cc3183590dce72059af6b6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166766
Reviewed-by: Patrick Luby <guibomacdev@gmail.com>
Tested-by: Jenkins
(cherry picked from commit 05d3a99aa687ee4e1706f9403651379b7ebdad89)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166723
Tested-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>

vcl/skia/gdiimpl.cxx

diff --git a/vcl/skia/gdiimpl.cxx b/vcl/skia/gdiimpl.cxx
index d7028b9..dce3175 100644
--- a/vcl/skia/gdiimpl.cxx
+++ b/vcl/skia/gdiimpl.cxx
@@ -1490,6 +1490,26 @@ void SkiaSalGraphicsImpl::invert(basegfx::B2DPolygon const& rPoly, SalInvert eFl
            aPaint.setShader(
                aBitmap.makeShader(SkTileMode::kRepeat, SkTileMode::kRepeat, SkSamplingOptions()));
        }

#ifdef SK_METAL
        // tdf#153306 prevent subpixel shifting of X coordinate
        // HACK: for some unknown reason, if the X coordinate of the
        // path's bounds is more than 1024, SkBlendMode::kExclusion will
        // shift by about a half a pixel to the right with Skia/Metal on
        // a Retina display. Weirdly, if the same polygon is repeatedly
        // drawn, the total shift is cumulative so if the drawn polygon
        // is more than a few pixels wide, the blinking cursor in Writer
        // will exhibit this bug but only for one thin vertical slice at
        // a time. Apparently, shifting drawing a very tiny amount to
        // the left seems to be enough to quell this runaway cumulative
        // X coordinate shift.
        if (isGPU())
        {
            SkMatrix aMatrix;
            aMatrix.set(SkMatrix::kMTransX, -0.001);
            getDrawCanvas()->concat(aMatrix);
        }
#endif
    }
    getDrawCanvas()->drawPath(aPath, aPaint);
    postDraw();