A regression of 1166966e.

We need to consider 2 use-cases:

1. Protecting an existing document with a password (by overwriting
   the original file). In this case we don't want to generate a
   thumbnail from the now encrypted file, but we do want to erase
   the stored thumbnail and show a generic icon instead.

2. Closing a modified document without saving the changes. Here we
   don't want to generate a thumbnail either, because it may
   contain the unsaved changes, but either we don't want to replace
   the stored thumbnail, because most likely it correctly represents
   the last saved state of the document.

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

Change-Id: I65a60acd956768f6a023c83053700803e5a8f1f2

Change-Id: Ia0b22e62001cff4a63ea197b77aebb1759f73122
Reviewed-on: https://gerrit.libreoffice.org/16664Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: Iccabcf6df662c0c4814a4c4f20d690778799e049

Mostly in Calc, but also for the SwAnnotationShell.
Also replace the current code in SwDrawTextShell with a call
to EditView::ChangeFontSize. The latter is able to handle
text with different sizes out of the box.

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

This suggests that the GF_FONTSHIFT thing is very broken as
its level 0 for all the font renderers here, but follow
the working pattern for now

Change-Id: Ia180a40071157ead9a3b04c05658a6574dd5f864

... if the process exits before the "bt" command, because then gdb will
print an error that there is no stack and stop the script before "quit".

Change-Id: I0f3a1381e17dda7e8a7d958e0cbc96676bbc23ce

Change-Id: I1c4f5e468062ced7f76b877939f96435b00d05e3

Change-Id: I516c8e7a33d0d05a619971b387352f8360d4765c

Change-Id: I76456d5e9a687ccf2f8b8cb9f64802de866206a8
Reviewed-on: https://gerrit.libreoffice.org/16635Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>

wcscpy_s is buffer size, not string length, so size must also account
for a terminating NULL

Change-Id: Ie69692e47c142f3b733b093d0b87fa4bb03db7b4

Change-Id: I74c683cf1ca821733717348a7840feeb7fbff3e9

Change-Id: I76ced60151bcc160eb4a4c12cc86f01edec076fc

Change-Id: Ie770ecceb8c8f5a6fa882a9f5d5a26806b029589

The buffered overlay manager paints using a timer, which is problematic
if the given vcl::Window already supports double-buffering itself, so
always use direct (to the rendercontext) painting in that case.

Change-Id: I93144c02814fd511f333224ab058374c7da369f0

Double buffering default is false, then can be enabled on a per-widget
basis, finally this can be overriden at runtime by the
VCL_DOUBLEBUFFERING_FORCE_ENABLE environment variable (so that
everything is painted using double buffering).

Let SupportsDoubleBuffering() also respect this variable, so code
calling SupportsDoubleBuffering() can react to the runtime override,
too.

Change-Id: Ic9a1c00a801f6976069d7cfc47c3fa491ebc1ff0

Change-Id: I08c60396d66a15ee94c7010753540c4e7fb3a0db

cf. <https://gerrit.libreoffice.org/#/c/16357> "Revert the removal of cast to
build on old poppler"

Change-Id: I75184cec64b4f32e0cf9adedeb84c8591be4fc73

Change-Id: I0ea809a888187624261182552cf7fa0a9c96c648

Change-Id: I0dd118f0215e06fec0cccff9b46d80f13bd802cc
Reviewed-on: https://gerrit.libreoffice.org/16620Reviewed-by: Matthew Francis <mjay.francis@gmail.com>
Tested-by: Matthew Francis <mjay.francis@gmail.com>

Change-Id: I44e2881bcf1d50fbc4d35802d90ea8ee19b39f3a

... instead of just creating a new document.

Change-Id: Ib336c07f6827775a9c1a4fa0b8fbfe4ae0ff8221

Change-Id: If5960be7b04db418a1a42bd7b661391f65e79498
Signed-off-by: Michael Stahl <mstahl@redhat.com>

Change-Id: I9309dc797c63e6c28831f545fd4d029a4d108dea

mbNoFocusRects is also the case now for gtk3, but this unfortunate
heuristic makes the database and chart wizards and various other
things just way too tall. This is likely a desperate effort to
make pre layout dialogs size to something sensible

Change-Id: I501f04ad96063935dd1f162e009ffd5053a820c4

Change-Id: I44eae735b710332d87f2f642911e988c796457cd

Change-Id: I22e9eacfddff611d368963162024437f4a7b7fdf

... to move test code out of production code

Change-Id: I8bd38f57faba41a0af3740eda098144d9e49fe15

With this, if e.g. Ctrl-A is hit for a hello-world document, then
selection is properly visible.

Change-Id: If3fae1e73dc76c50e62f6e1f31d99873636f0e93

Change-Id: I5df89581796eca4e48e0a305d4319e4674f8dd87

Change-Id: I5a990643903946ea339136fbc514ca8cb90fa34e

Change-Id: I86d2daec6dcbd6af3a8da1b8329417bae46c8079

See the comment 45 in the bug for the agreed design.

Change-Id: I428f339df48f530f606525434a4a2be8a79acaab

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

themeing has changed around quite a bit, and there's insufficient interest to
get older versions looking right

Change-Id: Ia2ee58d66c859e3bf00b54413fac1efb00d38dd3

This reverts commit 09b4cbe9,
no good reason to revert the original commit in the first place.

Conflicts:
	bridges/test/java_remote/Bug107753_Test.java

Change-Id: I6d3d50887460fd15080c46782c29a653c3684bec

This reverts commit 262dbabe,
no good reason to revert the original commit in the first place.

Conflicts:
	linguistic/source/lngsvcmgr.cxx

Change-Id: Ifa1896da3ec3fd291a9b30a9a6734bdcce406b99

...from most rtl/bootstrap.h functions.  They are effectively only called from
C++ code (there is no plain C UNO binding), so it should be fine to let std
exceptions (like bad_alloc or length_error) propagate from their implementations
to call sites.

(The exception is rtl_bootstrap_args_close, which is typically called from C++
dtors, so should not throw anyway.)

This would strictly speaking be an [API CHANGE], but it should make no practical
difference whether a process terminates abruptly because an exception cannot
pass through a SAL_THROW_EXTERN_C() nothrow specification or because legacy
client code does not expect exceptions to be thrown from functions from which
SAL_THROW_EXTERN_C() has now been removed.

Change-Id: I08e8479e9c5731e46021aadd6a725c1793024d10

Change-Id: I9c941cfcb8c856f35ff3b24f8e15000af4993651

Remove TReportPairHelper and TStartMarkerHelper as well since they
were helper structs that were used solely for many of these
o3tl::compose calls and are now unused.

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