Revert "tdf#114076: Expand ClipRange to next PixelBound" ...
and "tdf#124272 use ClipRegion's geometry if not a rectangle"
This commit reverts c1230cede19ae3633e51c7ca780cb34d9dbaa20f
and 362c1cf2bd580f6dc8bf27bdcd79174111bc1b5c
tdf#114076, or any of its duplicates, is not reproducible in master
if c1230cede19ae3633e51c7ca780cb34d9dbaa20f
("tdf#114076: Expand ClipRange to next PixelBound") is reverted.
(Tested on Linux and Windows)
So, if we revert it, we no longer need
362c1cf2bd580f6dc8bf27bdcd79174111bc1b5c
("tdf#124272 use ClipRegion's geometry if not a rectangle"), which
was a follow-up fix for a regression introduced by the first commit.
This also fixes tdf#129085 and all the duplicates, which were
introduced by the follow-up commit.
I plan to add the unittests in a different commit
Change-Id: Ie4328c15b24b521127c1b653bd621bfc92ac39cf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112340
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
diff --git a/sw/source/core/doc/notxtfrm.cxx b/sw/source/core/doc/notxtfrm.cxx
index ca834fd..dd93b15 100644
--- a/sw/source/core/doc/notxtfrm.cxx
+++ b/sw/source/core/doc/notxtfrm.cxx
@@ -973,60 +973,10 @@ void paintGraphicUsingPrimitivesHelper(
if(0 != aClip.count())
{
// tdf#114076: Expand ClipRange to next PixelBound
// Do this by going to basegfx::B2DRange, adding a
// single pixel size and using floor/ceil to go to
// full integer (as needed for pixels). Also need
// to go back to basegfx::B2DPolyPolygon for the
// creation of the needed MaskPrimitive2D.
// The general problem is that Writer is scrolling
// using blitting the unchanged parts, this forces
// this part of the scroll to pixel coordinate steps,
// while the ViewTransformation for paint nowadays has
// a sub-pixel precision. This results in an offset
// up to one pixel in radius. To solve this for now,
// we need to expand to the next outer pixel bound.
// Hopefully in the future we will someday be able to
// stay on the full available precision, but this
// will need a change in the repaint/scroll paradigm.
const basegfx::B2DRange aClipRange(aClip.getB2DRange());
const basegfx::B2DVector aSinglePixelXY(rOutputDevice.GetInverseViewTransformation() * basegfx::B2DVector(1.0, 1.0));
const basegfx::B2DRange aExpandedClipRange(
floor(aClipRange.getMinX() - aSinglePixelXY.getX()),
floor(aClipRange.getMinY() - aSinglePixelXY.getY()),
ceil(aClipRange.getMaxX() + aSinglePixelXY.getX()),
ceil(aClipRange.getMaxY() + aSinglePixelXY.getY()));
// create the enclosing rectangle as polygon
basegfx::B2DPolyPolygon aTarget(basegfx::utils::createPolygonFromRect(aExpandedClipRange));
// tdf#124272 the fix above (tdf#114076) was too rough - the
// clip region used may be a PolyPolygon. In that case that
// PolyPolygon would have to be scaled to mentioned PixelBounds.
// Since that is not really possible geometrically (would need
// more some 'grow in outside direction' but with unequal grow
// values in all directions - just maaany problems
// involved), use a graphical trick: The topology of the
// PolyPolygon uses the standard FillRule, so adding the now
// guaranteed to be bigger or equal bounding (enclosing)
// rectangle twice as polygon will expand the BoundRange, but
// not change the geometry visualization at all
if(!rOutputDevice.GetClipRegion().IsRectangle())
{
// double the outer rectangle range polygon to have it
// included twice
aTarget.append(aTarget.getB2DPolygon(0));
// add the original clip 'inside' (due to being smaller
// or equal). That PolyPolygon may have an unknown number
// of polygons (>=1)
aTarget.append(aClip);
}
rContent.resize(1);
rContent[0] =
new drawinglayer::primitive2d::MaskPrimitive2D(
aTarget,
aClip,
rContent);
}
}
