drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx | 99 +++++++++++++
drawinglayer/source/processor2d/vclmetafileprocessor2d.hxx | 3
2 files changed, 102 insertions(+)
New commits:
commit 846b18650c54e60a0e30b99c8980e39ee56c589f
Author: Armin Le Grand (Collabora) <armin.le.gr...@me.com>
AuthorDate: Tue Jun 17 17:59:02 2025 +0200
Commit: Armin Le Grand <armin.le.gr...@me.com>
CommitDate: Wed Jun 18 13:49:21 2025 +0200
tdf#166709 special handling of transparency for metafile/PDF
For handling transparency for tiled bitmaps correctly in
metafiles the newly added linear transparency is too much
and leading to many single transprent bitmnaps. This is
not wrong but too much data and bloating the metafile/PDF.
For details and extensive explanations see text in commit.
Change-Id: I255db614e0c9eb8cd5130abbdaef4eda1c398684
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/186630
Tested-by: Jenkins
Reviewed-by: Armin Le Grand <armin.le.gr...@me.com>
diff --git a/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
b/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
index de51c5ba6d61..8a35c34b4e0d 100644
--- a/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
+++ b/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
@@ -64,6 +64,7 @@
#include <drawinglayer/primitive2d/epsprimitive2d.hxx>
#include <drawinglayer/primitive2d/structuretagprimitive2d.hxx>
#include <drawinglayer/primitive2d/objectinfoprimitive2d.hxx> // for
Title/Description metadata
+#include <drawinglayer/primitive2d/fillgraphicprimitive2d.hxx>
#include <drawinglayer/converters.hxx>
#include <basegfx/matrix/b2dhommatrixtools.hxx>
#include <tools/vcompat.hxx>
@@ -899,6 +900,12 @@ void VclMetafileProcessor2D::processBasePrimitive2D(const
primitive2d::BasePrimi
static_cast<const
primitive2d::ObjectInfoPrimitive2D&>(rCandidate));
break;
}
+ case PRIMITIVE2D_ID_FILLGRAPHICPRIMITIVE2D:
+ {
+ processFillGraphicPrimitive2D(
+ static_cast<const
primitive2d::FillGraphicPrimitive2D&>(rCandidate));
+ break;
+ }
case PRIMITIVE2D_ID_TEXTHIERARCHYEMPHASISMARKPRIMITIVE2D:
{
// EmphasisMarks are traditionally not added to Metafiles, see
@@ -960,6 +967,98 @@ void VclMetafileProcessor2D::processObjectInfoPrimitive2D(
}
}
+void VclMetafileProcessor2D::processFillGraphicPrimitive2D(
+ primitive2d::FillGraphicPrimitive2D const& rFillGraphicPrimitive2D)
+{
+ // tdf#166709 check if we have to make an exception handling this
+ // FillGraphicPrimitive2D. If it
+ // - has transparency
+ // - is tiled
+ // - is a Bitmap
+ // - is not animated
+ // - is no embedded SVG
+ // we have to, see below
+ if (!basegfx::fTools::equalZero(rFillGraphicPrimitive2D.getTransparency(),
0.0))
+ {
+ // we have transparency
+ const attribute::FillGraphicAttribute&
rAttribute(rFillGraphicPrimitive2D.getFillGraphic());
+
+ if (rAttribute.getTiling())
+ {
+ // we have tiling
+ const Graphic& rGraphic(rAttribute.getGraphic());
+
+ if (GraphicType::Bitmap == rGraphic.GetType() &&
!rGraphic.IsAnimated()
+ && !rGraphic.getVectorGraphicData())
+ {
+ // tdf#166709 it is a Bitmap, not animated & not
+ // embedded SVG.
+
+ // conditions are met. Unfortunately for metafile
+ // and for PDF export we *need* all tiles which are
+ // potentially created by the decomposition of the
+ // FillGraphicPrimitive2D to be embedded to a single
+ // UnifiedTransparencePrimitive2D that holds that
+ // transparency - as it was before adding more
+ // possibilities for direct unified transparency.
+
+ // Despite the decomposition being correct and creating
+ // now BitmapAlphaPrimitive2D with every one holding the
+ // correct alpha, the 'old' way with encapsulating to
+ // a UnifiedTransparencePrimitive2D is needed here
+ // to create a single bitmap representation that then
+ // gets used. When not doing this a potentially high
+ // number of BitmapAlphaPrimitive2D will be exported,
+ // which is not an error but needs too much ressources,
+ // prevents loading of the created PDF for some viewers
+ // and bloats the PDF file.
+
+ // NOTE: I thought if only doing this for the PDF export
+ // case would make sense, but all exports still based
+ // on metafile potentially have this problem, so better
+ // do it in general at metafile creation already.
+
+ // NOTE: This shows how urgent it would be to create a
+ // PDF export using a PrimitiveRenderer instead of
+ // Metafile - that could do the right thing and use
+ // a representation in the PDF that is capable of
+ // tiling. No chance to do that with the existing
+ // metafile stuff we have.
+
+ // NOTE: The creation of the possible MetafileAction
+ // for this is done here locally in method
+ // processUnifiedTransparencePrimitive2D. Use that
+ // directly if necessary.
+
+ // So: create a FillGraphicPrimitive2D without transparency
+ // embedded to a UnifiedTransparencePrimitive2D representing
+ // the transparency and process it directly
+ rtl::Reference<primitive2d::BasePrimitive2D> aPrimitive(
+ new primitive2d::UnifiedTransparencePrimitive2D(
+ primitive2d::Primitive2DContainer{
+
rtl::Reference<primitive2d::FillGraphicPrimitive2D>(
+ new primitive2d::FillGraphicPrimitive2D(
+
rFillGraphicPrimitive2D.getTransformation(),
+ attribute::FillGraphicAttribute(
+ rGraphic, rAttribute.getGraphicRange(),
+ rAttribute.getTiling(),
rAttribute.getOffsetX(),
+ rAttribute.getOffsetY()))) },
+ rFillGraphicPrimitive2D.getTransparency()));
+
+ // process UnifiedTransparencePrimitive2D primitive directly
+ processUnifiedTransparencePrimitive2D(
+ static_cast<const
primitive2d::UnifiedTransparencePrimitive2D&>(*aPrimitive));
+
+ // we are done, return
+ return;
+ }
+ }
+ }
+
+ // all other cases: process recursively with original primitive
+ process(rFillGraphicPrimitive2D);
+}
+
void VclMetafileProcessor2D::processGraphicPrimitive2D(
const primitive2d::GraphicPrimitive2D& rGraphicPrimitive)
{
diff --git a/drawinglayer/source/processor2d/vclmetafileprocessor2d.hxx
b/drawinglayer/source/processor2d/vclmetafileprocessor2d.hxx
index e85df18f9e20..46d1cb2f1edf 100644
--- a/drawinglayer/source/processor2d/vclmetafileprocessor2d.hxx
+++ b/drawinglayer/source/processor2d/vclmetafileprocessor2d.hxx
@@ -65,6 +65,7 @@ class MaskPrimitive2D;
class UnifiedTransparencePrimitive2D;
class TransparencePrimitive2D;
class ObjectInfoPrimitive2D;
+class FillGraphicPrimitive2D;
class StructureTagPrimitive2D;
}
@@ -146,6 +147,8 @@ private:
const primitive2d::TransparencePrimitive2D& rTransparenceCandidate);
void
processObjectInfoPrimitive2D(const primitive2d::ObjectInfoPrimitive2D&
rObjectInfoPrimitive2D);
+ void processFillGraphicPrimitive2D(
+ primitive2d::FillGraphicPrimitive2D const& rFillGraphicPrimitive2D);
void processStructureTagPrimitive2D(
const primitive2d::StructureTagPrimitive2D& rStructureTagCandidate);
