diff options
author | Miklos Vajna <vmiklos@collabora.com> | 2020-09-18 17:52:41 +0200 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2020-09-22 09:00:27 +0200 |
commit | ef9b9de57c42ef91c47b1362804bd2be7ba1ae09 (patch) | |
tree | d3ef94a615900c3e10e27d3f3adf3b3a564aaf0a /emfio | |
parent | bfb559d04a08d2a8ebb90d04772c0321e2aaf1dc (diff) |
[API CHANGE] tdf#136836 emfio: set size hint on inner PDF if used as shape fill
The bugdoc has a shape, its bitmap fill is an EMF, which is actually a
PDF. The PDF is has a height of 5cm, but the shape has a height of 14
cm.
Inform vcl::RenderPDFBitmaps() about the size of the shape, so the
result won't be blurry. This approach makes sure that we don't
unconditionally render at higher resolution, i.e. the "load a PDF of 100
pages into Online" use-case won't use more memory than before.
API CHANGE, because the EMF reader is only available via UNO, though
it's likely that no actual external code would ever invoke it directly.
(cherry picked from commit 01024ee24c6e89044c68051f6fd5f1264905e90c)
Conflicts:
emfio/qa/cppunit/emf/EmfImportTest.cxx
include/vcl/pdfread.hxx
include/vcl/vectorgraphicdata.hxx
vcl/source/gdi/vectorgraphicdata.cxx
Change-Id: If1d8def0136d408a31a0cc54777a7f26430a0ff3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103101
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Diffstat (limited to 'emfio')
-rw-r--r-- | emfio/CppunitTest_emfio_emf_test.mk | 15 | ||||
-rw-r--r-- | emfio/inc/emfreader.hxx | 2 | ||||
-rw-r--r-- | emfio/qa/cppunit/emf/EmfImportTest.cxx | 60 | ||||
-rw-r--r-- | emfio/qa/cppunit/emf/data/pdf-in-emf.pptx | bin | 0 -> 35443 bytes | |||
-rw-r--r-- | emfio/source/emfuno/xemfparser.cxx | 15 | ||||
-rw-r--r-- | emfio/source/reader/emfreader.cxx | 4 |
6 files changed, 79 insertions, 17 deletions
diff --git a/emfio/CppunitTest_emfio_emf_test.mk b/emfio/CppunitTest_emfio_emf_test.mk index 4f892b42c27b..07e10df675ec 100644 --- a/emfio/CppunitTest_emfio_emf_test.mk +++ b/emfio/CppunitTest_emfio_emf_test.mk @@ -37,20 +37,7 @@ $(eval $(call gb_CppunitTest_use_libraries,emfio_emf,\ $(eval $(call gb_CppunitTest_use_ure,emfio_emf)) $(eval $(call gb_CppunitTest_use_vcl,emfio_emf)) -$(eval $(call gb_CppunitTest_use_components,emfio_emf,\ - configmgr/source/configmgr \ - dtrans/util/mcnttype \ - emfio/emfio \ - framework/util/fwk \ - i18npool/util/i18npool \ - package/source/xstor/xstor \ - package/util/package2 \ - toolkit/util/tk \ - sfx2/util/sfx \ - ucb/source/core/ucb1 \ - ucb/source/ucp/file/ucpfile1 \ - unotools/util/utl \ -)) +$(eval $(call gb_CppunitTest_use_rdb,emfio_emf,services)) $(eval $(call gb_CppunitTest_use_configuration,emfio_emf)) diff --git a/emfio/inc/emfreader.hxx b/emfio/inc/emfreader.hxx index 75a77211ea61..c6caa0fb9f26 100644 --- a/emfio/inc/emfreader.hxx +++ b/emfio/inc/emfreader.hxx @@ -34,6 +34,7 @@ namespace emfio bool mbEMFPlusDualMode : 1; /// An other format is read already, can ignore actual EMF data. bool mbReadOtherGraphicFormat = false; + basegfx::B2DTuple maSizeHint; bool ReadHeader(); // reads and converts the rectangle @@ -47,6 +48,7 @@ namespace emfio void ReadGDIComment(sal_uInt32 nCommentId); /// Parses EMR_COMMENT_MULTIFORMATS. void ReadMultiformatsComment(); + void SetSizeHint(const basegfx::B2DTuple& rSizeHint) { maSizeHint = rSizeHint; } private: template <class T> void ReadAndDrawPolyPolygon(sal_uInt32 nNextPos); diff --git a/emfio/qa/cppunit/emf/EmfImportTest.cxx b/emfio/qa/cppunit/emf/EmfImportTest.cxx index 94b4f6acd11f..6ad3e8972956 100644 --- a/emfio/qa/cppunit/emf/EmfImportTest.cxx +++ b/emfio/qa/cppunit/emf/EmfImportTest.cxx @@ -12,6 +12,12 @@ #include <test/bootstrapfixture.hxx> #include <test/xmltesttools.hxx> +#include <unotest/macros_test.hxx> + +#include <com/sun/star/frame/Desktop.hpp> +#include <com/sun/star/drawing/XDrawPagesSupplier.hpp> +#include <com/sun/star/beans/XPropertySet.hpp> +#include <com/sun/star/graphic/XGraphic.hpp> #include <comphelper/seqstream.hxx> #include <comphelper/sequence.hxx> @@ -22,38 +28,65 @@ #include <drawinglayer/primitive2d/baseprimitive2d.hxx> #include <drawinglayer/tools/primitive2dxmldump.hxx> +#include <comphelper/processfactory.hxx> #include <memory> namespace { +using namespace css; using namespace css::uno; using namespace css::io; using namespace css::graphic; using drawinglayer::primitive2d::Primitive2DSequence; using drawinglayer::primitive2d::Primitive2DContainer; -class Test : public test::BootstrapFixture, public XmlTestTools +class Test : public test::BootstrapFixture, public XmlTestTools, public unotest::MacrosTest { + uno::Reference<uno::XComponentContext> mxComponentContext; + uno::Reference<lang::XComponent> mxComponent; + void checkRectPrimitive(Primitive2DSequence const & rPrimitive); void testWorking(); void TestDrawString(); void TestDrawStringTransparent(); void TestDrawLine(); + void TestPdfInEmf(); Primitive2DSequence parseEmf(const OUString& aSource); public: + void setUp() override; + void tearDown() override; + uno::Reference<lang::XComponent>& getComponent() { return mxComponent; } + CPPUNIT_TEST_SUITE(Test); CPPUNIT_TEST(testWorking); CPPUNIT_TEST(TestDrawString); CPPUNIT_TEST(TestDrawStringTransparent); CPPUNIT_TEST(TestDrawLine); + CPPUNIT_TEST(TestPdfInEmf); CPPUNIT_TEST_SUITE_END(); }; +void Test::setUp() +{ + test::BootstrapFixture::setUp(); + + mxComponentContext.set(comphelper::getComponentContext(getMultiServiceFactory())); + mxDesktop.set(frame::Desktop::create(mxComponentContext)); +} + +void Test::tearDown() +{ + if (mxComponent.is()) + mxComponent->dispose(); + + test::BootstrapFixture::tearDown(); +} + Primitive2DSequence Test::parseEmf(const OUString& aSource) { const Reference<XEmfParser> xEmfParser = EmfTools::create(m_xContext); @@ -157,6 +190,31 @@ void Test::TestDrawLine() assertXPath(pDocument, "/primitive2D/metafile/transform/polypolygonstroke/line", "width", "33"); } +void Test::TestPdfInEmf() +{ + // Load a PPTX file, which has a shape, with a bitmap fill, which is an EMF, containing a PDF. + OUString aURL = m_directories.getURLFromSrc("emfio/qa/cppunit/emf/data/pdf-in-emf.pptx"); + getComponent() = loadFromDesktop(aURL); + + // Get the EMF. + uno::Reference<drawing::XDrawPagesSupplier> xDrawPagesSupplier(getComponent(), uno::UNO_QUERY); + uno::Reference<drawing::XDrawPage> xDrawPage(xDrawPagesSupplier->getDrawPages()->getByIndex(0), uno::UNO_QUERY); + uno::Reference<beans::XPropertySet> xShape(xDrawPage->getByIndex(0), uno::UNO_QUERY); + uno::Reference<graphic::XGraphic> xGraphic; + xShape->getPropertyValue("FillBitmap") >>= xGraphic; + Graphic aGraphic(xGraphic); + + // Check the size hint of the EMF, which influences the bitmap generated from the PDF. + const std::shared_ptr<VectorGraphicData>& pVectorGraphicData = aGraphic.getVectorGraphicData(); + + // Without the accompanying fix in place, this test would have failed with: + // - Expected: 14321 + // - Actual : 0 + // i.e. there was no size hint, the shape with 14cm height had a bitmap-from-PDF fill, the PDF + // height was only 5cm, so it looked blurry. + CPPUNIT_ASSERT_EQUAL(14321.0, pVectorGraphicData->getSizeHint().getY()); +} + CPPUNIT_TEST_SUITE_REGISTRATION(Test); } diff --git a/emfio/qa/cppunit/emf/data/pdf-in-emf.pptx b/emfio/qa/cppunit/emf/data/pdf-in-emf.pptx Binary files differnew file mode 100644 index 000000000000..61b2af28c8b3 --- /dev/null +++ b/emfio/qa/cppunit/emf/data/pdf-in-emf.pptx diff --git a/emfio/source/emfuno/xemfparser.cxx b/emfio/source/emfuno/xemfparser.cxx index f28633a73446..6e43788fcb83 100644 --- a/emfio/source/emfuno/xemfparser.cxx +++ b/emfio/source/emfuno/xemfparser.cxx @@ -37,6 +37,7 @@ #include <basegfx/matrix/b2dhommatrixtools.hxx> #include <unotools/ucbstreamhelper.hxx> #include <drawinglayer/primitive2d/metafileprimitive2d.hxx> +#include <sal/log.hxx> #include <wmfreader.hxx> #include <emfreader.hxx> @@ -59,6 +60,7 @@ namespace emfio { private: uno::Reference< uno::XComponentContext > context_; + basegfx::B2DTuple maSizeHint; protected: public: @@ -72,6 +74,7 @@ namespace emfio const uno::Reference< ::io::XInputStream >& xEmfStream, const OUString& aAbsolutePath, const uno::Sequence< ::beans::PropertyValue >& rProperties) override; + void SAL_CALL setSizeHint(const geometry::RealPoint2D& rSize) override; // XServiceInfo virtual OUString SAL_CALL getImplementationName() override; @@ -145,7 +148,9 @@ namespace emfio if (nMetaType == 0x464d4520) { // read and get possible failure/error, ReadEnhWMF returns success - bReadError = !emfio::EmfReader(*pStream, aMtf).ReadEnhWMF(); + emfio::EmfReader aReader(*pStream, aMtf); + aReader.SetSizeHint(maSizeHint); + bReadError = !aReader.ReadEnhWMF(); } else { @@ -215,12 +220,18 @@ namespace emfio } else { - OSL_ENSURE(false, "Invalid stream (!)"); + SAL_WARN("emfio", "Invalid stream (!)"); } return comphelper::containerToSequence(aRetval); } + void XEmfParser::setSizeHint(const geometry::RealPoint2D& rSize) + { + maSizeHint.setX(rSize.X); + maSizeHint.setY(rSize.Y); + } + OUString SAL_CALL XEmfParser::getImplementationName() { return XEmfParser_getImplementationName(); diff --git a/emfio/source/reader/emfreader.cxx b/emfio/source/reader/emfreader.cxx index 133b6ccf7f83..2811bcec7e2e 100644 --- a/emfio/source/reader/emfreader.cxx +++ b/emfio/source/reader/emfreader.cxx @@ -505,6 +505,10 @@ namespace emfio return; } + // aGraphic will be the only output of the EMF parser, so its size hint can be the same as + // ours. + aGraphic.getVectorGraphicData()->setSizeHint(maSizeHint); + maBmpSaveList.emplace_back(new BSaveStruct(aGraphic.GetBitmapEx(), aOutputRect, SRCCOPY)); const std::shared_ptr<VectorGraphicData> pVectorGraphicData = aGraphic.getVectorGraphicData(); |