Change-Id: I4962d739af889595f026c250b8f10a917b1dfb17
Reviewed-on: https://gerrit.libreoffice.org/32130Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
Tested-by: Takeshi Abe <tabe@fixedpoint.jp>

bt:
1  0x00002aaade2fea5b in accessibility::AccessibleGridControlTableBase::ensureIsValidIndex (this=0x5555586abdb0, nChildIndex=11)
    at /home/julien/lo/libreoffice/accessibility/source/extended/AccessibleGridControlTableBase.cxx:257
2  0x00002aaade2f97f2 in accessibility::AccessibleGridControlTable::getAccessibleChild (this=0x5555586abdb0, nChildIndex=11)
    at /home/julien/lo/libreoffice/accessibility/source/extended/AccessibleGridControlTable.cxx:60
3  0x00002aaade2ee054 in accessibility::AccessibleGridControl::commitTableEvent (this=0x555557e97500, _nEventId=5, _rNewValue=uno::Any(void), _rOldValue=uno::Any(void))
    at /home/julien/lo/libreoffice/accessibility/source/extended/AccessibleGridControl.cxx:354

Seems a regression from https://cgit.freedesktop.org/libreoffice/core/commit/?id=0f32ba56a1e775e8ecd6c827ebd48f7f2fb51c81

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

Change-Id: I85e4c411548768a9a85686aab35e5795116f0308

At least with SAL_USE_VCLPLUGIN=gtk, sticking DBG_TESTSOLARMUTEX() before and
after this while loop demonstrates that, without the explicit
SolarMutexReleaser, the SolarMutex was acquired before the loop but isn't
thereafter.

Change-Id: I4a97063103193821cc5eb3469484dd3346a60822

Change-Id: I53bfa5ef7594eeb7446cdb9e4ba211839e255197

Change-Id: Id22412389f3de2c9923887fd99a3e1c6860e1f33

Change-Id: I45b57f6789a6156b7f70b39bac624bab55dc17ff

Change-Id: Ib710f1ee291fb15ad7b05a5d36824a129d407ccc

Change-Id: I783b2950fcd43b1767961bc97d331be96b035ba7

Change-Id: I1b4d5fe33859cdabdcaa159a489e019ff8fe88b6

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

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

Change-Id: Idd25c574e24bdab06474ff13661e0c5d121a9992
Reviewed-on: https://gerrit.libreoffice.org/32080Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: David Tardon <dtardon@redhat.com>

Change-Id: Idac32c3d788714533ee760782d2b6a328262f3f8
Reviewed-on: https://gerrit.libreoffice.org/31996Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

This seems to be a race condition that is not really related to the UI
testing. It should happen any time with out-of-process UNO.

Change-Id: I86e42fc6c3848c4522e6f4e5db72e92917e19286
Reviewed-on: https://gerrit.libreoffice.org/31995Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

Change-Id: Ic86e7e75ac8667b32227683c5b48731831203a0f
Reviewed-on: https://gerrit.libreoffice.org/31994Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

sw comments are handled by editeng, so tables in comments aren't
imported. That means lcl_emptyRow() may get a row where the first cell's
start is an empty reference, so handle that case.

Change-Id: I43e286314143934c20e6cbbd656d32883b6b2efe

Add boolean field "duplicate": true

Change-Id: I8e45ed4694bfc75b18c2405146e5eaa142559c1b
Reviewed-on: https://gerrit.libreoffice.org/31997Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Akshay Deep <akshaydeepiitr@gmail.com>

Change-Id: Iba67eb8dc0cb40e8663ad04fc96201072d3aeabe

...at least with --disable-pch it complained

> [build CXX] canvas/source/directx/dx_bitmapcanvashelper.cxx
> C:/lo/core/include\vcl/vclptr.hxx(98): error C2027: use of undefined type 'OutputDevice'
> c:\lo\core\canvas\source\directx\dx_devicehelper.hxx(33): note: see declaration of 'OutputDevice'
> c:\lo\core\canvas\source\directx\dx_devicehelper.hxx(109): note: see reference to class template instantiation 'VclPtr<OutputDevice>' being compiled

Change-Id: I3ba90acdb632fc209b8137818e30fb771ba0807c

Change-Id: I0732f23dd9daa0c3a1f138011b0db17b36b729d1
Reviewed-on: https://gerrit.libreoffice.org/32179Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

cf. 86e6d9b6 "Make check for broken static
initialize_list work with MSVC 2015"

Change-Id: Iac957a7bcaeecf9740e53ab777a032d128dfffde

New standard palette; contribution from Christoph Schaefer, freieFarben e.V.

Change-Id: I36cfb3983e28caad0994f9a81e5bc230e8a675e8
Reviewed-on: https://gerrit.libreoffice.org/32116Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: V Stuart Foote <vstuart.foote@utsa.edu>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: I025847785cef1cad9190861410a284c848744dce

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

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

Change-Id: I6ee4993ec65ba60234daf3e6fcc4cab432939f5f

... and ensure that they are removed from the mpShapeMap.

Commit 76c549eb added all sub-shapes of
group shapes to SwAccessibleMap::mpShapeMap.
It's not immediately obvious why this is necessary, given that
AccessibleShape will probably create these on demand.

It turns out that the AccessibleShapes are not only disposed from
SwAccessible code but also from svx code,
in particular the call to ViewForwarderChanged() in
SwAccessibleContext::ChildrenScrolled() leads to visibility checks.

Thus the RemoveGroupContext() may fail to remove the sub-shapes from the
mpShapeMap and the assertion in ~SwAccessibleMap is triggered.

Change-Id: I6d9c3d816f5521960855d5e6a563bec49d8030b8

Change-Id: I0aa18fafd1759cd7c4d2ada18120fa8b97b353a4

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

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

...where AC_RUN_IFELSE failed to find libucrt.lib

Change-Id: If696224d179f5e59d22f0a2eaa417ebe80a0ed8a

Change-Id: I4663de7eb81b7af39b5a24581667fd7fcfb829cd

Change-Id: Iaef197e9ab736c219897933a76f4c75a3e3acaa6

This reverts commit d3c06fb9.

which seems innocent after all

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

LibreOffice's default paragraph style is very different from Word's normal
style? Anyway, if normal is not defined in styles.xml, then don't
re-assign any other automatic styles to be based off of it, since that
indicates that the style information being imported is not complete.

avoids regression from commit b79b5e0d

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

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

Change-Id: I56e68ab19e63637d21dea7929f093bb1aceb982a
Reviewed-on: https://gerrit.libreoffice.org/32135Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>

ImplImageTree was used outside of VCL which is not consistent with
the name and the header also contains a lot of implementation
detail. This separates the implementation to ImplImageTree and
the public interface and singleton to ImageTree only.

Change-Id: I3a26444f0f6971a6b1d83472e9cef19c93192d3e
Reviewed-on: https://gerrit.libreoffice.org/32134Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>