1) Fixed the trailing space issue in text decoration attribute value.

2) Changed the SVG export suite in order to make it more reusable for
other tests.

3) Added a unit test for testing that text decorations are exported
correctly.

Change-Id: I2bd71974242a0007726fbdd5ef5637a9ec62fd47
Reviewed-on: https://gerrit.libreoffice.org/19870Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
(cherry picked from commit 50506421)
Reviewed-on: https://gerrit.libreoffice.org/20259Tested-by: Andras Timar <andras.timar@collabora.com>

Project: translations  36951384710f07e1d233ffb702441123701dfaae

Updated Slovenian translation

Change-Id: I1059cbd5c4acd6d33dec1f39799fcdb9f36dffcf

As for the change in chart2/qa/extras/chart2export.cxx
Chart2ExportTest::testAxisNumberFormatXLSX() unit test: also Excel
writes string parts of format codes quoted, including minus sign in
negative subformat.

Change-Id: I201bb012df818129cbc65de0eee8eca59e57d829
(cherry picked from commit 509cfa40)

Change-Id: I4d5f8aa33fffceaa080d8b2ef6a177b4680cf761
(cherry picked from commit 7340872a)

Taking implementation from sc/source/filter/excel/xestyle.cxx
GetNumberFormatCode(), slightly modified to ensure valid conversion and
force en-US locale data. Also don't unnecessarily convert if format is
for system locale and system locale is en-US.

Change-Id: I9223eaa655132b4106a35c94cb0005559d7575b1
(cherry picked from commit 2011b541)

... Excel doesn't seem to care though.

Change-Id: I3697a808d8fee2417f0b0e03dba2b94ceea133dd
(cherry picked from commit ea1db935)

Change-Id: I83b0cd033ba46dabdde92ed2812398518d1895bf
(cherry picked from commit 2246f478)

... to conflate the places that do this on their own.

Change-Id: Idde2173780e0515ad982b4be46fc4df23a7577ad
(cherry picked from commit b5554804)

Change-Id: I6f999f36d6ef57dadddd774e6ef60ceb9cb8033c
(cherry picked from commit 9affe65c)

This is the same problem that

commit 133e04fc
Author: Caolán McNamara <caolanm@redhat.com>
Date:   Wed Jun 17 09:23:32 2015 +0100

    Resolves: tdf#91880 Invalidate graphics when the gtk window is destroyed

    not just when the GtkSalFrame is dtored

tried to fix, but that just made it more unlikely to fail

Change-Id: Icba750c787adb6cd5c5ed0874ef07e6201c4cf25
(cherry picked from commit 26c32cfe)

Change-Id: I25c6fe1bd1df568911838ab161a130f8b0571e16
(cherry picked from commit 5b9062a2)

content.cxx loops over CONTEXT_COUNT so there can't be holes here

regression from

commit 63aa32b7
Author: Caolán McNamara <caolanm@redhat.com>
Date:   Mon Sep 28 17:05:40 2015 +0100

    move strings out of widget

Change-Id: Ie189e2a910e02a96bc8e1450a2f2b03afaf466ac
(cherry picked from commit b33236bf)

Change-Id: Ia4f4def1fd58d82a7c994114e45a36227269671c
(cherry picked from commit fa7708de)

Change-Id: I054786a97b6f0e6ba3fe3b824ff9ff9748d8be4a
Reviewed-on: https://gerrit.libreoffice.org/20013Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit 33a774bb)
Signed-off-by: Eike Rathke <erack@redhat.com>

Change-Id: Iab196e87e0eb4b063d77fe4af10c58b8355665c4
Reviewed-on: https://gerrit.libreoffice.org/20210Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: Ib2042d6419dafcd62aaec95efb844cb09833c266

Change-Id: I8bf7f8d6ba1a54f2969e055956fadc52ff6c8673

Change-Id: Ibeb1f6627815fc34c6e166357c88e076b75f6abb
Reviewed-on: https://gerrit.libreoffice.org/20197Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: Ia95c4893773bec77fe2ddf1a85e8687ab6776e10

Change-Id: I94975d220452ca91cbbd9db5e7895fa0b3a88e4a

Change-Id: Iee56131a15f9554c263406356456f65694515719

Fixes the busy-loop - 100% CPU all the time.

Change-Id: I965f62d6a6f2ec1830c0897dd97179a739c70afc
Reviewed-on: https://gerrit.libreoffice.org/20186Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: I6f7d7d3b5b027097417a15804a42aaaab4a03158
Reviewed-on: https://gerrit.libreoffice.org/20185Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: I1423de820ec40340d2ddb9030106b813e488f2f3
Reviewed-on: https://gerrit.libreoffice.org/20184Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

"errorOccured" is an English spelling mistake, but is the
name of the event, which cannot / should not be changed for
backwards compatibility reasons.

Spelling was boguously "corrected" in:

  commit 60a9b517
  Author: Takeshi Abe <tabe@fixedpoint.jp>
  Date:   Sat Dec 4 12:56:38 2010 +0900

      Replace all occured, occurance etc.

This was leading to impossibility of assigning a macro
to that event, since it was not appearing in this dialog.

Change-Id: Ib0373501cba8af8f4abb1c62faddf8d65c926a7e

49c2b980 changed a binary test doc

Change-Id: I77cd86ce3eccfc701f54630870045e04244ac048

ah!, the original code had a type in it. That's
why it was refactored to remove the null check.

Now a proper fix for cids: 1326180<->1326190

Change-Id: Iba7fd47c03eb5c157f878e0e297e8688f20ae348
(cherry picked from commit 8244fc26)
Reviewed-on: https://gerrit.libreoffice.org/20179Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: I30ea6cea81ae1b4f344e253c7a51cfef4d844128
(cherry picked from commit f4ac7854)
Reviewed-on: https://gerrit.libreoffice.org/20178Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: Ibd1c15ce65501d4dc6d0cea8c8e5f4e683d1b7a4
(cherry picked from commit c5bfac3f)
Reviewed-on: https://gerrit.libreoffice.org/20177Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: I9f76340ccdb32f4337d0ebe85e9957fe8a3b19f9
(cherry picked from commit 43e2b417)
Reviewed-on: https://gerrit.libreoffice.org/20176Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: Id984cc1f5954be38fbe3a1a15bfe811d7cdfe99d
(cherry picked from commit d05ae550)
Reviewed-on: https://gerrit.libreoffice.org/20175Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Noto fonts correctly.

LanguageTag()::getLanguage() would always return "zh" despite
traditional Chinese were choose. hb_language_from_string()
requires iso639-3 language tag except for Chinese, so
we use use Bcp47() language tag for Chinese to differ zh-CN and
the others.

Change-Id: I6700a79a9c49a97d840b430def77741912935f72
Reviewed-on: https://gerrit.libreoffice.org/20027Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
(cherry picked from commit 4424daa2)
Reviewed-on: https://gerrit.libreoffice.org/20203

Excel expects this order, so let XclExpTables be managed as
XclExpRecordBase in the sheet's XclExpRecordList.

Change-Id: If2cefc255c74688661e861a26218564117b1e3ce
(cherry picked from commit 4112ecad)

Change-Id: I433ea34ed036be9ee4810b6db75a2dbb945b23a2

Change-Id: I591532d6a0dd07f8c621ecea3eaa4b92f2225493

Change-Id: I1fb5baabd869c4abe8ccf9eeae131efcc697bbee

Otherwise we get workbook load events (eg.) happening at idle, which
occurs at (more or less) random places as the StarBasic interpreter
Yields every few instructions executed.

Change-Id: I51ea57f4ad45e30e04fe84a9dc051ed1c5537536

"(abc)|(def)" matches "def" with result offset sequences
{0,-1,0},{3,-1,3}

And thus the assert() calls introduced with
4cf1d290 and
ce91f3c1 were hit.

Change-Id: I571b6c7d47349a2cc7b1d1e34193b2865012a49f
(cherry picked from commit 4dd2d406)

Change-Id: I2d1185d433be0fbb57a59e0609ab0f505248a348
Reviewed-on: https://gerrit.libreoffice.org/19210Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
(cherry picked from commit d65d9c54)
Reviewed-on: https://gerrit.libreoffice.org/20170

The old code in MultiSalLayout::GetTextBreak() only makes sense if the
base level character width is zero where a fallback level has a
non-zero character width, and vice versa. But this is not the case for
Windows, at least not any more now when using UniscribeLayout and not
SimpleWinLayout.

This simple change fixes that: Only use the width from a fallback
level if the width at the base level is zero. Hopefully it does not
cause regressions for other documents or on other platforms. (But, I
repeat, I find it hard to believe that the intent of the code could
really have been to ever add two or more non-zero widths for the same
character from different levels of fallback and use that.)

Change-Id: Ic66c55db4b7463f9e04fcedec76f1c44f5e62e03
(cherry picked from commit a0d112a5)