Given the css stuff a given style can end up with itself as a grandparent but
then the css mutates things so it doesn't loop around to give itself as a
grandgrandparent.

So undo the loop detection check and turn it into a parse depth test
instead with an arbitrary, but high, level

Change-Id: If28684ad2b2adc664bc2da6b176b93248b377162
Reviewed-on: https://gerrit.libreoffice.org/59813
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

No reason for SalGraphics::getSystemDependentDataManager() to be inside
the #if HAVE_FEATURE_OPENGL. Fixes build for iOS.

Change-Id: I3d8174a4841d793f8d7ea2c3ebf3be2888229a3e

Change-Id: I2600a59e4eac7375882aff0491ecf16bad449520
Reviewed-on: https://gerrit.libreoffice.org/59841
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>

Change-Id: I61b0d754caddb659d3bed7fb7321bfba9a6123d7
Reviewed-on: https://gerrit.libreoffice.org/59819
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: I21cca2e5cd81ee58a59fda077ed40ba02d9f1df3
Reviewed-on: https://gerrit.libreoffice.org/59844
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: Ia16bde5b428be6a39ff7e3f6052258652ea0f07a
Reviewed-on: https://gerrit.libreoffice.org/59789
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

Change-Id: I6ce82c6f061c7f01a5584230c26e9bf20c7f07ca
Reviewed-on: https://gerrit.libreoffice.org/59788
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

...see d84bf553 "Silence external/boost
deprecated std::iterator warnings with MSVC 2017"

Change-Id: I886ce27b90184aa0044f5a89f40cdd69507a70c4
Reviewed-on: https://gerrit.libreoffice.org/59831
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

...when MSVC creates ExplicitCategoriesProvider's implicit copy functions (and
--disable-pch doesn't happen to include the definition of XLabeledDataSequence
anyway).  Easiest fix is to explicitly delete those (apparently unused) copy
functions.

Change-Id: I983a7f20729ba066b4fa5342800382d830492848
Reviewed-on: https://gerrit.libreoffice.org/59830
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

Caller of WriteAnimationNodeCommonPropsStart always
pass true for parameter bSingle.

Change-Id: I780a09e2fb34b3040561d4c2b437f8c4cc23df4a
Reviewed-on: https://gerrit.libreoffice.org/59800
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

Change-Id: Icd685d903d12eb867fe0545e605d2389f40d6e70
Reviewed-on: https://gerrit.libreoffice.org/59799
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

Change-Id: I28810ca142f9b4dcc4bf76305c7fa4e3504adf3b
Reviewed-on: https://gerrit.libreoffice.org/59837
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: I39ebede67761bd77027975cc7991190dae88e7ad
Reviewed-on: https://gerrit.libreoffice.org/59765
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

Create a new PPTXAnimationExport and move animation
related member functions from PowerPointExport.

Change-Id: Iaeb9ff032c248d9b540680a4937365004d0c921e
Reviewed-on: https://gerrit.libreoffice.org/59735
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

Change-Id: Ifec76dcd5d6cccefe9e1d0d939d799fcde88b087
Reviewed-on: https://gerrit.libreoffice.org/59812
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

Change-Id: I7992bee3cf88c670d932cd6d6873339ac5f0c104
Reviewed-on: https://gerrit.libreoffice.org/59817
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

Added unit test for Added fix for
   Change-Id: Ic1f173c85d3824afabb5b7ebf3a8594311eb9007

Change-Id: I38444587d00b96d52ff725dc7c5852e057bc6bd9
Reviewed-on: https://gerrit.libreoffice.org/59828
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

Change-Id: Ifb1b5b30e1c3586be75ff01f7bc0612a6b8b1f8d
Reviewed-on: https://gerrit.libreoffice.org/59785
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

to get rid of function pointers.

Change-Id: Iaea4782274c655484559118183397a3340d1d7cb
Reviewed-on: https://gerrit.libreoffice.org/59734
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

Change-Id: I836766e308fb198f4f081837731ba1854c770f06
Reviewed-on: https://gerrit.libreoffice.org/59733
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

of PowerPointExport to anonymous namespace. This allows cleaning
up PowerPointExport and later moving other non-static member
functions caller outside PowerPointExport to become a new class.

Change-Id: I174337896db68307d7859dc9cf8bb9a9648f90eb
Reviewed-on: https://gerrit.libreoffice.org/59732
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

Change-Id: Icbe68ac5f1ddb5ad59e0615ad1d558b829c6977f
Reviewed-on: https://gerrit.libreoffice.org/59832
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
Tested-by: Maxim Monastirsky <momonasmon@gmail.com>

Toolbar popup windows have the toolbar as their direct
parent. This causes problems when such window is teared-off
and later closed, as Window::dispose will attempt to move the
focus to its parent which is the toolbar, instead of what
users expect to have the focus back in the document area, or
in whatever was the last focused control. As a solution
reparent the window before dispose, similar to what happens
with floating toolbars.

Also return the focus to the main window when popup mode ends,
so we can keep track of the last focused control. This is also
a good thing by itself, as WB_OWNERDRAWDECORATION windows don't
get the focus by default, which results in neither the floating
window nor the document have focus after tear-off.

Change-Id: I060b8c45a64db9c612da58b7c35478bab41a4558
Reviewed-on: https://gerrit.libreoffice.org/59811
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>

Change-Id: I2d13401dc21548018235bab715d54eb3ae81b51b
Reviewed-on: https://gerrit.libreoffice.org/59807
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

This got lost when SvNumberFormatter::INPUTSTRING_PRECISION was
introduced but not all places changed using the previous hard
coded number 300, so input of fraction of second in time clock and
duration was either truncated or not available at all, already
since

    commit 0ce0dd58
    CommitDate: Tue Dec 1 16:58:07 2009 -0500

        #i46511# Fixed one bug where the standard percent format
        incorrectly got unlimited precision.

Change-Id: I9e4dd867b07090db16b23639fd01fb2cebb3f5d0
Reviewed-on: https://gerrit.libreoffice.org/59815Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins

This is a first step to allow buffering of system
dependent data, especially (but not only) for the
system-dependent implementations of graphic output.
For example, for B2DPolygon and Win output, it allows
buffering the Gdiplus::GraphicsPath instead of re-
creating it all the time.
To support that, the change includes forwarding the
current transformation to the renderers in SalGraphics.
The current state in VCL is to transform all and
everything to device coordinates at every single
paint.
I have currently started to do this for ::drawPolyLine
implementations. The fallbacks for all systems will
at the start of that method just transform the data
to device coordinates, so all works as before.
This may also be done for FilledPolygon paint in a later
step, but most urgent is FatLine painting.
An arrangement of shared_ptr/weak_ptr is used so that
either the instance buffering (in the example B2DPolygon)
or the instance managing it can delete it. The instance
managing it currently uses a 1s Timer and a cycle-lifetime
management, but that can be extended in the future
to e.g. include size hints, too.
The mechanism it designed to support multiple Data per
buffering element, e.g. for B2DPolygon at the same time
system-dependent instances of Gdiplus and Cairo can be
buffered, but also PDF-data.
This is achieved semi-automatic by using
typeid(class).hash_code() as key for organization.
The mechanism will be used for now at B2DPolygon, but
is not limited to. There is already a similar but less
general buffer (see GdiPlusBuffer) that can and will
be converted to use this new mechanism.

Added vcl/headless Cairo renderer to support given
ObjectToDevice transformation (not to transform given
B2DPolygon)
Added support for CairoPath buffered at B2DPolygon,
seems to work well. Need to do more tests

Moved usage to templates suggested by Noel Grandin
(Noel Grandin <noelgrandin@gmail.com>), thanks for
these suggestions. Adapted Win usage to that, too.

Converted Win-specific GdiPlus BitmapBuffer to new
mechanism, works well. Checked, the manager holds
now a mix of bitmap and path data under Win

Added a cleanup mechanism to flush all buffered data
at DeInitVCL() using flushAll() at
SystemDependentDataBuffer

Adapted Linux-versions of ::drawPolyLine to support
PixelSnapHairline, for now in a simplified version
that still allows buffering. This will also be used
(and use buffering) for the Cairo-fallback in
X11SalGraphics

Change-Id: I88d7e438a20b96ddab7707050893bdd590c098c7
Reviewed-on: https://gerrit.libreoffice.org/59555Tested-by: Armin Le Grand <Armin.Le.Grand@cib.de>
Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>

Move animation export functions in PowerPointExport to
a single file for further refactor work.

Change-Id: I957599bb293fcdda3d51db48334ef5825b1cba64
Reviewed-on: https://gerrit.libreoffice.org/59731
Tested-by: Jenkins
Reviewed-by: Mark Hung <marklh9@gmail.com>

Also remove SlideSorter::CreateController which only calls new;
fix a memory leak in osl_Security's MyTestPlugInImpl::initialize

Change-Id: I70b6e888984f8543adbf879162e752556d2b3f0e
Reviewed-on: https://gerrit.libreoffice.org/59805Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>

I suppose if language is set, that happens in the wrapping ReqIF
document, not in the XHTML fragments.

Change-Id: I970c3810b7f02bbafc37dd85e095b254cc651330
Reviewed-on: https://gerrit.libreoffice.org/59810
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>

It is possible that we not only have a dummy OLE object, but both the
replacement image and the native data is missing. Handle these enough so
that we don't give up exporting the document and also produce valid
output.

Change-Id: Ia973ecfbfb7e8fdecdc831b29d3a82c988ed7ea5
Reviewed-on: https://gerrit.libreoffice.org/59806Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins

Change-Id: Ia658776bce8744e7b771559ca12b94a20923fc52
Reviewed-on: https://gerrit.libreoffice.org/59782
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: Ib7d79f9ea2192e798891ea1c2a9d547c8ddf3574
Reviewed-on: https://gerrit.libreoffice.org/59790
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>

Change-Id: Ibcee9a6aab0b04bf52f3e75a46f52d98864eee4c
Reviewed-on: https://gerrit.libreoffice.org/59808
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

This allows to avoid the "(Visual Studio 2015)" labels next to project
names in Solution Explorer; and also avoid multiple warnilgs like

    project-name.vcxproj : warning  : The build tools for Visual Studio
    2015 (v140) cannot be found. Install Visual Studio 2015 (v140) to
    build using the Visual Studio 2015 (v140) build tools.

in IDE's Output pane in case VS2015 is absent locally.

Change-Id: I53b68ac810cbf2a31667c35dd549310e7209e010
Reviewed-on: https://gerrit.libreoffice.org/59797Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>

Change-Id: Ia078fb8e1e497edfa08e2a61d1659100461fc52e
Reviewed-on: https://gerrit.libreoffice.org/59720
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

Change-Id: I86f14a38c32baf46ccb5674005192fea2665719d
Reviewed-on: https://gerrit.libreoffice.org/59801
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

This reverts commit a191075e.

This fix was superceded by commit 13a05eec
which is better abstracted.

Change-Id: I3b8ba735d279a50f5548a26fcc8fa6f790b2ea42

...focus manager.

This prevents escape key press on More Options tool box in some panel
title bars from sending focus to doc view.

Change-Id: Iaac9bf31494e4e35f30350580acaa9f42ead49a0
Reviewed-on: https://gerrit.libreoffice.org/59783
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>

The absence of "on=" is treated in Word (tested 2003) as off.

Change-Id: Ibc6b0e5ca0f25a9c3ca1b9505fa24c4d821bfd0
Reviewed-on: https://gerrit.libreoffice.org/59457
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>

- Fedora 28 has python3-markdown2 package containing /usr/bin/markdown2-3
- the "alias" doesn't work at all because it's not exported to the
  sub-shell in $(markdown) so use variable instead

Change-Id: Icf2af1e66750269e0660f3897a91734cadf5d978