summaryrefslogtreecommitdiff
path: root/svgio
AgeCommit message (Collapse)AuthorFilesLines
2016-04-15SVG: Add unittest for ecc7f698b5f080530f006218fa3dd82da43d9abbXisco Fauli2-0/+43
Change-Id: Icbfd60bd15196649011919e8dea064276e1964b0 Reviewed-on: https://gerrit.libreoffice.org/24097 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-14loplugin:passstuffbyref in variousNoel Grandin1-1/+1
Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
2016-04-13tdf#94306 remove unused boost dependenciesJochen Nitschke1-1/+0
remove <boost/noncopyable.hpp> in pch and remove boost from makefile if it was the only boost entry. Change-Id: Icb945ae59c137571f4f63807601738eea5c3e831 Reviewed-on: https://gerrit.libreoffice.org/24061 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-04-13tdf#94306 replace boost::noncopyable in stoc to xmlsec..Jochen Nitschke3-7/+11
Replace with C++11 delete copy-constructur and copy-assignment. Remove boost/noncopyable.hpp includes. Add missing default ctors. With this commit there should be no users of boost::noncopyable left. Change-Id: I6b1e47824912a6a80cc3f00f34938ebc048d8975 Reviewed-on: https://gerrit.libreoffice.org/24051 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-04-12SVG: Add unittest for tdf#47446 and tdf#98599Xisco Fauli3-0/+71
Change-Id: I86bec5694eb0c85596d2573111ddb355bd3ae09e Reviewed-on: https://gerrit.libreoffice.org/24011 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-12SVG: Check that masking-path-07-b.svg doesn't hangXisco Fauli2-0/+161
Change-Id: If8b9a0dd955611f8ebb2859fa9b10a61bfb2230b Reviewed-on: https://gerrit.libreoffice.org/24010 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-12SVG: actually this code is unnecessary after...Xisco Fauli1-13/+1
... f5d4a688346a939a58b1df69d771dede177b3370 Change-Id: I2911763c38a6be275bcd7ae0c9a51cd7a2e9fe4a Reviewed-on: https://gerrit.libreoffice.org/24012 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-12SVG: Add unittest for #i125329#Xisco Fauli2-0/+38
Change-Id: I2d6fbd81c8d95283cf8937739e338ddac14d8df0 Reviewed-on: https://gerrit.libreoffice.org/24008 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-12SVG: Clean up this code a litle bitXisco Fauli2-7/+6
Change-Id: Ib941321d43a9a04d57f3c7c4a55528927fed5036 Reviewed-on: https://gerrit.libreoffice.org/24009 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-04tdf#97499 Fixed containers parameters clearing #4tymyjan2-4/+2
Change-Id: I7c96181399f4d7e62d4aceca404b22d68f903513 Reviewed-on: https://gerrit.libreoffice.org/23754 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-04tdf#97499 Fixed containers parameters clearing #3tymyjan1-3/+2
Change-Id: I1c105de7e1df251533179fba77e3797ef1c4e2d4 Reviewed-on: https://gerrit.libreoffice.org/23752 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-03SVG: Using the marker property from a style sheet is equivalent...Xisco Fauli1-2/+5
... to using all three (start, mid, end). However, shorthand properties cannot be used as presentation attributes. Change-Id: Ie3842b8fa87f562f038600f8f012d8434363b3cd Reviewed-on: https://gerrit.libreoffice.org/23749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-01tdf#97966 Drop 'static' keywordsWastack1-1/+1
Including no keywords from extern "C" blocks Change-Id: I364aa7d67c2bf867588798e07df95c3f3f2bd7f3 Reviewed-on: https://gerrit.libreoffice.org/23676 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-30tdf#97657: SVGIO: Don't iterate over parent's clip-paths...Xisco Fauli1-2/+7
... when it's a clip-path content Change-Id: I222b429a61409d9670b9d60357895542a15afc15 Reviewed-on: https://gerrit.libreoffice.org/23624 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-29SVGIO: Don't inherit style attributes from parent if the...Xisco Fauli3-108/+54
... shapes define their own attributes. Partially revert commit c2d130425cc876cceb96cc7e47636426a9e48fa8 Change-Id: Iee98003f4ff63575a28818b091ab77431337192f Reviewed-on: https://gerrit.libreoffice.org/23571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-26SVGIO: Shapes within a clippath element might inherit...Xisco Fauli3-0/+92
... fill, stroke, stroke-width and others attributes from parent element, like use element Change-Id: I911bc50409428e81135b81e6dfc82acd1c21d113 Reviewed-on: https://gerrit.libreoffice.org/23527 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-17tdf#98599 SVG: consider attributes of 'defs' elementRegina Henschel1-2/+2
The element 'defs' is not rendered, but children of it may inherit the attributes. Therefore these has to be considered. Change-Id: Ic6c21645439731db63f22adf5a5a4e2af2b74b71 Reviewed-on: https://gerrit.libreoffice.org/23309 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-03-11tdf#47446: SVGIO: Take markers' parents into account tooXisco Fauli1-1/+13
Change-Id: I41b7228ab5980581ab5e1c8c4d4d466da6045086 Reviewed-on: https://gerrit.libreoffice.org/23124 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-10tdf#97539: SVGIO: iterate over parent's clippathsXisco Fauli1-1/+2
Change-Id: Iadb335bf5b328fa3fd9683da5433c299e62afa8e Reviewed-on: https://gerrit.libreoffice.org/23106 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-10Extract Directories from BootstrapFixtureBaseStephan Bergmann1-2/+2
(as some tests derive from the latter only for the Directories part, not for the setUp/tearDown overrides: those tests will be cleaned up next) Change-Id: Ib6b78eea868b8bc21d4cc6e8fd9e1d025deca05f Reviewed-on: https://gerrit.libreoffice.org/23078 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-09tdf#47446: SVGIO: Set marker's fill attribute to black...Xisco Fauli1-1/+1
... if it's not set Change-Id: Iefec59f885c3708defec2b636836617bf97387fc Reviewed-on: https://gerrit.libreoffice.org/23046 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-09SVGIO: Fix problem when style's parent contains a mask elementXisco Fauli2-10/+41
Change-Id: I26aab6c0f959f6cc78d040f59f1acf0c7e29931b Reviewed-on: https://gerrit.libreoffice.org/23045 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-08Updated to solve problem with gerrit patch 22363Jan Iversen1-1/+1
Regina posted comment on 22363, that this change was needed. Change-Id: I9021d40ee7c1ca7ca74d67c61fe58ce976da38c0
2016-03-07Revert "tdf#97538: SVGIO: Add unittest"Xisco Faulí2-46/+0
It fails in Tinderbox Linux-rpm_deb-x86@71-TDF This reverts commit 3c465c259c617e30b01c5ea25b5203a605cd4269. Change-Id: I20fc69dd785354cc6567b73251ee9161a811bdf0 Reviewed-on: https://gerrit.libreoffice.org/22987 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-07SVGIO: tdf#97539: clip-path elements might contain...Xisco Fauli2-10/+41
reference to other clip-paths Change-Id: I3722b31cefa4df6225e369b3d1db9f46be5933ff Reviewed-on: https://gerrit.libreoffice.org/22956 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-07tdf#97538: SVGIO: Add unittestXisco Fauli2-0/+46
Change-Id: I5bf523ac532ad19a005d99e19deb1e0ec9ce96e4 Reviewed-on: https://gerrit.libreoffice.org/22955 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-06SVGIO: Check minX, minY, maxX and maxY in polypolygonXisco Fauli1-0/+25
Change-Id: I768bef34ad0ba7708134e73981dabf0d47c2f259 Reviewed-on: https://gerrit.libreoffice.org/22952 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-03SVGIO: Check height and width of polypolygoncolorXisco Fauli1-0/+12
Change-Id: Ibc5844d4e4a89fae7bbc8d2b854384d0c433500f Reviewed-on: https://gerrit.libreoffice.org/22883 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-03tdf#98113: SVGIO: Add SvgANode class to handle transport ...Xisco Fauli6-0/+180
... attribute in anchor elements. I used Svggnode class a a reference for this. Change-Id: Id2a58bd913f9984dc91163ca0f333c016aa981f1 Reviewed-on: https://gerrit.libreoffice.org/22822 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-01loplugin:unuseddefaultparam in svgioNoel Grandin3-4/+4
Change-Id: I47fe88cd8c943acb9162cdfca53fb5fdbaacec04
2016-03-01fix typoXisco Fauli1-1/+1
Change-Id: I3e1e28269d8b2d6d45bba6bc43aa8f20c1963956 Reviewed-on: https://gerrit.libreoffice.org/22793 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2016-02-28tdf#98119 Allow markers on zero-length linesRegina Henschel1-11/+11
Inkscape, Seamonkey, Chrome and Batik show markers on zero- length lines, LibreOffice should do that too. The patch removes the restriction start<>end. Change-Id: I984baf755df7b2689e00d2a23b40e3f059c0add0 Reviewed-on: https://gerrit.libreoffice.org/22740 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2016-02-27Revert "SVGIO: tdf#97539: clip-path elements might contain..."Xisco Faulí2-18/+1
This reverts commit 7b7ffedfa6357b902a5e354d986dac39a3b2f462. Change-Id: I018830e17cb83c5fe3574b76939673877ac2004d Reviewed-on: https://gerrit.libreoffice.org/22726 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-25tdf#97936: SVGIO: Add unittestXisco Fauli2-0/+21
Change-Id: I7adb55dc408811f03365578f815bf1b7fbe7f2fd Reviewed-on: https://gerrit.libreoffice.org/22679 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-25Partially revert b6a62f07557fe4ae0931e0f3e0f306d2f69b116bXisco Fauli1-9/+6
it has been fixed by 423b79e7366203db3f57dea75b8cb9eb852b5614 in a better way Change-Id: Ia3192071f00522ac111d3e95e4dc05dc57a6267e Reviewed-on: https://gerrit.libreoffice.org/22677 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-21tdf#97538 no cropping if width, height and viewBox missingRegina Henschel2-38/+70
Now the whole image is inserted, so that it can be cropped in the application. The patch repairs in addition a wrong ratio in the case height is missing but viewBox is given, and fixes some comments. Change in SvgImportTest.cxx by Xisco Fauli. Change-Id: I8c0fbf8b78cf7ab5bc97d04783b49892da3d2849 Reviewed-on: https://gerrit.libreoffice.org/22363 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-19new loplugin: find write-only fieldsNoel Grandin3-9/+4
Change-Id: I0f83939babacf92485420ee63f290a297d7cb717 Reviewed-on: https://gerrit.libreoffice.org/22498 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-19SVGIO: Fix problem when relative font-size depends on its...Xisco Fauli3-0/+36
... parent's font-size Change-Id: I13943923c401a5a06b3aa51181fd75292534e888 Reviewed-on: https://gerrit.libreoffice.org/22452 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-18tdf#97752 SVGIO ignore not matching parent paintsJochen Nitschke1-6/+6
fill attributes can have ONE of 3 kinds of paints: * pain colour fills * gradient fills * pattern fills if element has a fill we should not get fills of an other kind from parent Change-Id: I376a8fea4a7185b037a069a28191b7f37f6788bd Signed-off-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-on: https://gerrit.libreoffice.org/22273 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-18SVGIO: tdf#97941: Don't double tspan fontsizeXisco Fauli4-0/+30
Same as in 701324a1e1f7e0c181ff1a50956ced686785ea53 Change-Id: I531bef4821008ef71951506c133f999b9ab4f4ff Reviewed-on: https://gerrit.libreoffice.org/22450 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-17SVGIO: tdf#97539: clip-path elements might contain...Xisco Fauli2-1/+18
reference to other clip-paths Change-Id: I0c426c892da2478293b78ea93db087d781275896 Reviewed-on: https://gerrit.libreoffice.org/22411 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-13SVGIO: tdf#97659: Add support for RGBAXisco Fauli6-15/+107
Change-Id: Icbf3796cdc95f91d298a5ca52d44931b3985eac6 Reviewed-on: https://gerrit.libreoffice.org/22303 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-11SVGIO: tdf#97543: Add import testXisco Fauli2-0/+19
Change-Id: Ia89f43ed1da9ead254f191bcbcc4ac338a9dda7a Reviewed-on: https://gerrit.libreoffice.org/22302 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-11tdf#97542: SVGIO: First check the current fill attributeXisco Fauli4-6/+71
Differenciate when gradient is defined in the text element or in its parents Add unit tests Change-Id: I85b09f72fbb1a3c8d2fa7350b4396b5bf924277f Reviewed-on: https://gerrit.libreoffice.org/22131 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-09SVGIO: tdf#79163: Add import testXisco Fauli2-0/+24
Change-Id: I891425f331e9be21c160eb0b11f05dba280bef23 Reviewed-on: https://gerrit.libreoffice.org/22196 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-09SVGIO: tdf#85770: Add import testXisco Fauli2-0/+24
Change-Id: Ifffe1d461ae6fa30a9cc61df2ece5716078031ab Reviewed-on: https://gerrit.libreoffice.org/22190 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-09SVGIO: tdf#45771: Add import testXisco Fauli2-0/+19
Change-Id: Ib1b66a3d91374bcf458bb368483f33cd4b8e0f0d Reviewed-on: https://gerrit.libreoffice.org/22189 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-09SVGIO: Add import test for 7c151f39144b7401a99c7b736a41e5dce4ac4db1Xisco Fauli2-0/+19
Change-Id: I4ab4463dfa2422fd0d8fa1fd65d7765f716c9bc3 Reviewed-on: https://gerrit.libreoffice.org/22188 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-02-09Remove excess newlinesChris Sherlock8-12/+0
A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-09SVGIO: Add import test for Fontsize keywordsXisco Fauli2-0/+38
Change-Id: I042a1deeec1c07f73f09ddd2fb71b9d2eba6b8c6 Reviewed-on: https://gerrit.libreoffice.org/22187 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Reviewed-by: jan iversen <jani@documentfoundation.org>