Change-Id: I5ed924d9d79db169b11a0561b89da9b57eb078d9
Reviewed-on: https://gerrit.libreoffice.org/66971Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>

Change-Id: I356871429f2e26d9d34e0e1428334a4d2287b683
Reviewed-on: https://gerrit.libreoffice.org/67295Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>

Fixes a regression after commit 5b2f1243

The destruction of the dialog is initiated by a callback executed from
VCLXButton::ProcessWindowEvent, which notifies listeners, using a call
to VCLXWindow::ImplExecuteAsyncWithoutSolarLock. The stack looks like
this:

ucrtbased.dll!issue_debug_notification(const wchar_t * const message) Line 28
	at minkernel\crts\ucrt\src\appcrt\internal\report_runtime_error.cpp(28)
ucrtbased.dll!__acrt_report_runtime_error(const wchar_t * message) Line 154
	at minkernel\crts\ucrt\src\appcrt\internal\report_runtime_error.cpp(154)
ucrtbased.dll!abort() Line 51
	at minkernel\crts\ucrt\src\appcrt\startup\abort.cpp(51)
comphelper.dll!comphelper::SolarMutex::doRelease(bool bUnlockAll) Line 65
	at c:\lo\src\core2\comphelper\source\misc\solarmutex.cxx(65)
vclplug_winlo.dll!SalYieldMutex::doRelease(bool bUnlockAll) Line 169
	at c:\lo\src\core2\vcl\win\app\salinst.cxx(169)
comphelper.dll!comphelper::SolarMutex::release(bool bUnlockAll) Line 89
	at c:\lo\src\core2\include\comphelper\solarmutex.hxx(89)
vcllo.dll!SalInstance::ReleaseYieldMutexAll() Line 114
	at c:\lo\src\core2\vcl\source\app\salvtables.cxx(114)
vcllo.dll!Application::ReleaseSolarMutex() Line 548
	at c:\lo\src\core2\vcl\source\app\svapp.cxx(548)
fps.dll!SolarMutexReleaser::SolarMutexReleaser() Line 1479
	at c:\lo\src\core2\include\vcl\svapp.hxx(1479)
fps.dll!fpicker::win32::vista::AsyncRequests::~AsyncRequests() Line 84
	at c:\lo\src\core2\fpicker\source\win32\asyncrequests.cxx(84)
fps.dll!fpicker::win32::vista::VistaFilePicker::~VistaFilePicker() Line 72
	at c:\lo\src\core2\fpicker\source\win32\vistafilepicker.cxx(72)
fps.dll!fpicker::win32::vista::VistaFilePicker::`scalar deleting destructor'(unsigned int)
cppuhelper3MSC.dll!cppu::OWeakObject::release() Line 233
	at c:\lo\src\core2\cppuhelper\source\weak.cxx(233)
cppuhelper3MSC.dll!cppu::WeakComponentImplHelperBase::release() Line 84
	at c:\lo\src\core2\cppuhelper\source\implbase.cxx(84)
fps.dll!cppu::PartialWeakComponentImplHelper<com::sun::star::ui::dialogs::XFilePicker3,com::sun::star::ui::dialogs::XFilePickerControlAccess,com::sun::star::ui::dialogs::XFilePreview,com::sun::star::ui::dialogs::XFolderPicker2,com::sun::star::lang::XInitialization,com::sun::star::lang::XServiceInfo>::release() Line 86
	at c:\lo\src\core2\include\cppuhelper\compbase.hxx(86)
invocationlo.dll!com::sun::star::uno::cpp_release(void * pCppI) Line 48
	at c:\lo\src\core2\include\com\sun\star\uno\genfunc.hxx(48)
cppu3.dll!cppu::_release(void * p, void(*)(void *) release) Line 86
	at c:\lo\src\core2\cppu\source\uno\prim.hxx(86)
cppu3.dll!cppu::_destructAny(_uno_Any * pAny, void(*)(void *) release) Line 130
	at c:\lo\src\core2\cppu\source\uno\destr.hxx(130)
cppu3.dll!uno_any_destruct(_uno_Any * pValue, void(*)(void *) release) Line 131
	at c:\lo\src\core2\cppu\source\uno\any.cxx(131)
invocationlo.dll!com::sun::star::uno::Any::~Any() Line 111
	at c:\lo\src\core2\include\com\sun\star\uno\any.hxx(111)
invocationlo.dll!stoc_inv::Invocation_Impl::~Invocation_Impl()
invocationlo.dll!stoc_inv::Invocation_Impl::`scalar deleting destructor'(unsigned int)
cppuhelper3MSC.dll!cppu::OWeakObject::release() Line 233
	at c:\lo\src\core2\cppuhelper\source\weak.cxx(233)
invocationlo.dll!stoc_inv::Invocation_Impl::release() Line 110
	at c:\lo\src\core2\stoc\source\invocation\invocation.cxx(110)
pyuno_d.pyd!com::sun::star::uno::Reference<com::sun::star::script::XInvocation2>::~Reference<com::sun::star::script::XInvocation2>() Line 110
	at c:\lo\src\core2\include\com\sun\star\uno\reference.hxx(110)
pyuno_d.pyd!pyuno::PyUNOInternals::~PyUNOInternals()
pyuno_d.pyd!pyuno::PyUNOInternals::`scalar deleting destructor'(unsigned int)
pyuno_d.pyd!pyuno::PyUNO_del(_object * self) Line 81
	at c:\lo\src\core2\pyuno\source\module\pyuno.cxx(81)
python35_d.dll!_Py_Dealloc(_object * op) Line 1795
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\object.c(1795)
python35_d.dll!dict_dealloc(PyDictObject * mp) Line 1646
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\dictobject.c(1646)
python35_d.dll!_Py_Dealloc(_object * op) Line 1795
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\object.c(1795)
python35_d.dll!subtype_dealloc(_object * self) Line 1194
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\typeobject.c(1194)
python35_d.dll!_Py_Dealloc(_object * op) Line 1795
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\object.c(1795)
python35_d.dll!frame_dealloc(_frame * f) Line 431
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\frameobject.c(431)
python35_d.dll!_Py_Dealloc(_object * op) Line 1795
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\object.c(1795)
python35_d.dll!fast_function(_object * func, _object * * * pp_stack, int n, int na, int nk) Line 4858
	at c:\lo\src\core2\workdir\unpackedtarball\python3\python\ceval.c(4858)
python35_d.dll!call_function(_object * * * pp_stack, int oparg) Line 4783
	at c:\lo\src\core2\workdir\unpackedtarball\python3\python\ceval.c(4783)
python35_d.dll!PyEval_EvalFrameEx(_frame * f, int throwflag) Line 3291
	at c:\lo\src\core2\workdir\unpackedtarball\python3\python\ceval.c(3291)
python35_d.dll!_PyEval_EvalCodeWithName(_object * _co, _object * globals, _object * locals, _object * * args, int argcount, _object * * kws, int kwcount, _object * * defs, int defcount, _object * kwdefs, _object * closure, _object * name, _object * qualname) Line 4080
	at c:\lo\src\core2\workdir\unpackedtarball\python3\python\ceval.c(4080)
python35_d.dll!PyEval_EvalCodeEx(_object * _co, _object * globals, _object * locals, _object * * args, int argcount, _object * * kws, int kwcount, _object * * defs, int defcount, _object * kwdefs, _object * closure) Line 4092
	at c:\lo\src\core2\workdir\unpackedtarball\python3\python\ceval.c(4092)
python35_d.dll!function_call(_object * func, _object * arg, _object * kw) Line 627
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\funcobject.c(627)
python35_d.dll!PyObject_Call(_object * func, _object * arg, _object * kw) Line 2166
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\abstract.c(2166)
python35_d.dll!method_call(_object * func, _object * arg, _object * kw) Line 330
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\classobject.c(330)
python35_d.dll!PyObject_Call(_object * func, _object * arg, _object * kw) Line 2166
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\abstract.c(2166)
python35_d.dll!PyEval_CallObjectWithKeywords(_object * func, _object * arg, _object * kw) Line 4634
	at c:\lo\src\core2\workdir\unpackedtarball\python3\python\ceval.c(4634)
python35_d.dll!PyObject_CallObject(_object * o, _object * a) Line 2092
	at c:\lo\src\core2\workdir\unpackedtarball\python3\objects\abstract.c(2092)
pyuno_d.pyd!pyuno::Adapter::invoke(const rtl::OUString & aFunctionName, const com::sun::star::uno::Sequence<com::sun::star::uno::Any> & aParams, com::sun::star::uno::Sequence<short> & aOutParamIndex, com::sun::star::uno::Sequence<com::sun::star::uno::Any> & aOutParam) Line 251
	at c:\lo\src\core2\pyuno\source\module\pyuno_adapter.cxx(251)
msci_uno.dll!`anonymous namespace'::callVirtualMethod(void * pAdjustedThisPtr, long nVtableIndex, void * pRegisterReturn, _typelib_TypeClass eReturnTypeClass, long * pStackLongs, long nStackLongs) Line 74
	at c:\lo\src\core2\bridges\source\cpp_uno\msvc_win32_intel\uno2cpp.cxx(74)
msci_uno.dll!`anonymous namespace'::cpp_call(bridges::cpp_uno::shared::UnoInterfaceProxy * pThis, bridges::cpp_uno::shared::VtableSlot aVtableSlot, _typelib_TypeDescriptionReference * pReturnTypeRef, long nParams, _typelib_MethodParameter * pParams, void * pUnoReturn, void * * pUnoArgs, _uno_Any * * ppUnoExc) Line 249
	at c:\lo\src\core2\bridges\source\cpp_uno\msvc_win32_intel\uno2cpp.cxx(249)
msci_uno.dll!unoInterfaceProxyDispatch(_uno_Interface * pUnoI, const _typelib_TypeDescription * pMemberDescr, void * pReturn, void * * pArgs, _uno_Any * * ppException) Line 430
	at c:\lo\src\core2\bridges\source\cpp_uno\msvc_win32_intel\uno2cpp.cxx(430)
invocadaptlo.dll!stoc_invadp::AdapterImpl::invoke(const _typelib_TypeDescription * pMemberType, void * pReturn, void * * pArgs, _uno_Any * * ppException) Line 467
	at c:\lo\src\core2\stoc\source\invocation_adapterfactory\iafactory.cxx(467)
invocadaptlo.dll!adapter_dispatch(_uno_Interface * pUnoI, const _typelib_TypeDescription * pMemberType, void * pReturn, void * * pArgs, _uno_Any * * ppException) Line 617
	at c:\lo\src\core2\stoc\source\invocation_adapterfactory\iafactory.cxx(617)
msci_uno.dll!`anonymous namespace'::cpp2uno_call(bridges::cpp_uno::shared::CppInterfaceProxy * pThis, const _typelib_TypeDescription * pMemberTypeDescr, _typelib_TypeDescriptionReference * pReturnTypeRef, long nParams, _typelib_MethodParameter * pParams, void * * pCallStack, __int64 * pRegisterReturn) Line 155
	at c:\lo\src\core2\bridges\source\cpp_uno\msvc_win32_intel\cpp2uno.cxx(155)
msci_uno.dll!`anonymous namespace'::cpp_mediate(void * * pCallStack, long nFunctionIndex, long nVtableOffset, __int64 * pRegisterReturn) Line 345
	at c:\lo\src\core2\bridges\source\cpp_uno\msvc_win32_intel\cpp2uno.cxx(345)
msci_uno.dll!`anonymous namespace'::cpp_vtable_call() Line 380
	at c:\lo\src\core2\bridges\source\cpp_uno\msvc_win32_intel\cpp2uno.cxx(380)
tklo.dll!ActionListenerMultiplexer::actionPerformed(const com::sun::star::awt::ActionEvent & evt) Line 146
	at c:\lo\src\core2\toolkit\source\helper\listenermultiplexer.cxx(146)
tklo.dll!ActionListenerMultiplexer::actionPerformed(const com::sun::star::awt::ActionEvent & evt) Line 146
	at c:\lo\src\core2\toolkit\source\helper\listenermultiplexer.cxx(146)
tklo.dll!VCLXButton::ProcessWindowEvent::__l8::<lambda>() Line 593
	at c:\lo\src\core2\toolkit\source\awt\vclxwindows.cxx(593)
tklo.dll!std::_Invoker_functor::_Call<void <lambda>(void) &>(VCLXButton::ProcessWindowEvent::__l8::void <lambda>(void) & _Obj)
tklo.dll!std::invoke<void <lambda>(void) &>(VCLXButton::ProcessWindowEvent::__l8::void <lambda>(void) & _Obj)
tklo.dll!std::_Invoker_ret<void,1>::_Call<void <lambda>(void) &>(VCLXButton::ProcessWindowEvent::__l8::void <lambda>(void) & <_Vals_0>)
tklo.dll!std::_Func_impl_no_alloc<void <lambda>(void),void>::_Do_call()
tklo.dll!std::_Func_class<void>::operator()()
tklo.dll!VCLXWindowImpl::OnProcessCallbacks(void * __formal) Line 300
	at c:\lo\src\core2\toolkit\source\awt\vclxwindow.cxx(300)
...

So it seems like the only place where we could guarantee the invariant
to hold solar mutex for GUI protection before releasing it is right
before the releaser itself.
Co-authored-by: Jan-Marek Glogowski <glogow@fbihome.de>

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

Used at least in the Edit > Edit Style... > Area > Color dialog for
the Pick button..

Change-Id: I864ec5918482a2f8ee5c5d80b79eacd6e8fcd88e

Change-Id: I2e7380dcf2af1a51c837a798a20de03d50eb7065
Reviewed-on: https://gerrit.libreoffice.org/70304
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

This reverts commit 84662e90.

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

Change-Id: I49e3a53db5137b62063f2664c945e256ced33856
Reviewed-on: https://gerrit.libreoffice.org/70300
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

Change-Id: I6d582032d419b832aa2610cad61d9c5596fcc9a4
Reviewed-on: https://gerrit.libreoffice.org/70299
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

In a previous commit only x/y coordinate were considered.

For better overview make use of the OOXML converter for orients and relations.

Change-Id: I9792ccfbc2ebb58fd768c14278cdfd9b54efe62f
Reviewed-on: https://gerrit.libreoffice.org/69523
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

Change-Id: I7224eaf1b569d018a85fee6a65afe43d0d37bfbc
Reviewed-on: https://gerrit.libreoffice.org/70298
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

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

This opacity influences the whole shape (stroke and fill).

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

Change-Id: Ie6a9ef6441b2b0053ea539bc0f1846fba8422313
Reviewed-on: https://gerrit.libreoffice.org/70297
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

I'm not exactly sure how, but in case
sw/qa/uitest/findReplace/findSimilarity.py and
sw/qa/uitest/findReplace/findReplace.py are in the same suite, then 3
tests from the later fail for me.

I don't see anything obviously problematic in either of these tests, so
just split them to separate suites, these are slow tests after all, so
more parallelism is good anyway.

(What may happen is that one of the dialogs remember its state in the
user profile, but then why this doesn't happen for everyone is unclear.)

Change-Id: If11f5b171972422fab5f79a047b1aabb1adee04d
Reviewed-on: https://gerrit.libreoffice.org/70296Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins

With this commit we are making numbers of columns
dynamic, but the number of maximum supported
columns will be the same (1024).
Such approach will allow us to check issues
(eg. performance, LO format etc.), and improve it.

Increasing number of maximum columns, will be done
in separate commit.

Change-Id: Ibac4101e9ffc05e3548eca1c198f6319ac7ff9aa
Reviewed-on: https://gerrit.libreoffice.org/44802
Tested-by: Jenkins
Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>

Sometimes, developers mention other bugs in the body as
related issues.
This might create mislead information in the release notes
E.g: https://wiki.documentfoundation.org/Releases/6.1.1/RC1
* Caolán is mentioned as fixer of i#66963 because the id is in
the body of 9c3d8b1c
which fixes tdf#98558
* M. Stahl is mentioned as fixer of tdf#112447 because the id is
in the body of 31e66bd0
This bug was fixed by Bjoern Michaelsen in
ce2fce9a

Change-Id: I8d0ee038741dbb27364a56a7fa9285f32839c37b
Reviewed-on: https://gerrit.libreoffice.org/69750
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>

Change-Id: I775e9d6d6bedbfe00a4e5349cf0a664ccbd0ecc4
Reviewed-on: https://gerrit.libreoffice.org/70288
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

So that Insert - Comment command would not insert an & on Windows
with Hungarian keyboard layout

Change-Id: I83f43cdc0b3f32e284cd747648de5bcb07505314
Reviewed-on: https://gerrit.libreoffice.org/70279
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>

Corresponding command was removed in
b94abbc8 (2004!)
This and 15bdac20
is why this accelerator showed up as uno:AnimationMode in the list
instead of a human readable name

Change-Id: I7b2f2cf76c75d815b85b1ceb6e8468588e3d15f1
Reviewed-on: https://gerrit.libreoffice.org/70268
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>

The uno:StyleCatalog command was removed in
b4c18dc0

Change-Id: I31118e97c5445b1021ea60b2629f12d2f60595cd
Reviewed-on: https://gerrit.libreoffice.org/70267
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>

Change-Id: Ie46fcc58039ce9686b779d27699e4988e93e8bd7
Reviewed-on: https://gerrit.libreoffice.org/70195
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

Commit 881043d1 (#i98792# Allow frequent
calls to slide show update()., 2009-04-27) added a comment in
sd::SlideshowImpl::updateSlideShow() talking about an "above" case, when
the slideshow update suggests to not wait till the next update,
but later commit 12dcf5e6 (slideshow:
cleanup main-loop usage, post-yield listeners, etc., 2015-11-13) removed
that handling.

That was fine till opengl rendering was introduced, which swaps its
backbuffer in an idle handler. Given that OpenGLFlushIdle's priority is
TaskPriority::POST_PAINT and SlideshowImpl::maUpdateTimer's priority is
TaskPriority::REPAINT, the GL backbuffer is not always painted while an
animation is ongoing.

This is more visible when tweaking the bugdoc to have an e.g. 5 sec
animation, then starting the slideshow results in waiting for 5 seconds
and only then the slide shows up, without an animation.

Fix the problem by re-introducing the handling of the fUpdate == 0, and
processing idle events there, which means the GL backbuffer will be
always painted, regardless if fUpdate is a small number or 0.

Change-Id: I25b2dc0aa41af62d9bd89617178eb2e9997f5b17
Reviewed-on: https://gerrit.libreoffice.org/70287Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins

Found with bin/find-unneeded-includes
Only removal proposals are dealt with here.

Change-Id: Ic2bd25bcbcc7f5fb6e29ced70fddb74385b5fb2f
Reviewed-on: https://gerrit.libreoffice.org/70077
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

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

These were originally added in commit
4105862b

Avoid the reported problem for the other two instances
(Calc and Master document) too

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

Found by:
run-clang-tidy-7 -checks=-*,misc-unused-using-decls

Change-Id: Iee4e9df37d4c971ea97256d041afbc3d3aaa3d27
Reviewed-on: https://gerrit.libreoffice.org/70027
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

Note that we can't work with the table box (and frame format) of
pTableTextNodeInfoInner in RtfAttributeOutput::TableVerticalCell(). This
is because cell definitions are written in
RtfAttributeOutput::TableDefinition(), and the loop only calls
setCell(), i.e. the cell index is updated in the node info, but the
table box is not.

Import was already working as-is.

Change-Id: Ia089c42b3a49f1a8f012a8cadaa8bcad16128c3e
Reviewed-on: https://gerrit.libreoffice.org/70269
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

... to take into account possible differences of results of dynamic_cast
vs static_cast; the change casts dynamically, as before the commit, and
only adds asserts on the result of the cast.

Thanks to sberg for pointing my mistake out!

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

Change-Id: Ib808e35d3cc3c2895993193ac6ed05d556c1857d
Reviewed-on: https://gerrit.libreoffice.org/70275
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>

Change-Id: I6b264a2f2e2995e880169f6224c60f023584469d
Reviewed-on: https://gerrit.libreoffice.org/70270
Tested-by: Jenkins
Reviewed-by: Andras Timar <andras.timar@collabora.com>

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

Change-Id: Ia6e48da3abe6d142135f0e916f02e27b63bbcb57
Reviewed-on: https://gerrit.libreoffice.org/70280
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>

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

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

These helpers are used in code generator code; so any invalid UTF-8
or UTF-16 in the conversion is programmer's error which needs fixing.
Thus, the behavior of toUtf8/fromUtf8 which asserts the validity is
fine here.

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

This reverts commit 50580f45.

Revert "raise cairo baseline to 1.10.0"

This reverts commit 58a0e60d.

32bit baseline is currently at cairo 1.8.8

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

there may have been changes not caught by undo to the object's
ordinal, so on use of RemoveObject (or ReplaceObject) call GetOrdNum
on the object to get its current position within its parent.

We can keep nOrdNum for the desired dest position within the parent
for InsertObject

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

In addition remove spinbox pressed state (it wasn't useful) and
draw white filled empty rect for spinbox as we don't draw the
background automatically.

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

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

The helper marco is used in export code to convert UTF-16 internal
string representation to UTF-8 used in XML encoding. I suppose that all
strings here should be already valid UTF-16 (an invalid input should
have been validated at import/input stage). An invalid string at this
stage means a programming error in another part of code that breaks this
precondition, and should not be handled at export stage.
(See also commit 0267a232.)

This effectively changes flags used in conversion from UTF-16 to UTF-8,
so that now RTL_UNICODETOTEXT_FLAGS_INVALID_ERROR is used instead of
RTL_UNICODETOTEXT_FLAGS_INVALID_DEFAULT, so that the result would be
truncated on invalid bytes instead of inserting U+FFFD, and would fail
assertion in debug builds.
Other changed flags don't affect UTF-16-to-UTF-8 conversion.

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