Change-Id: I6c57a52b4fc3ecb993af4526fefcca352ad269b5

Change-Id: I130c40227a756b396c4f858468457867d53454b1

Change-Id: I2441e15d2b32b897561cb404a925390299fea6d8

Change-Id: Ia0280d3d4f2d5b8a93a37a93847951391e928028

Removed bPrinter check from DrawTransformedBitmapEx, also removes
meOutDevType.

Change-Id: I5fd09efc4f3467702ab9671dc45da1a9c0e3f927
Reviewed-on: https://gerrit.libreoffice.org/8723Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

GetBitCount() works differently for VirtualDevices. GetAlphaBitCount()
is really only used by VirtualDevice, so moved functionality from
OutputDevice to VirtualDevice.

Change-Id: Ic00e32f1fa385542bcce8c9475f0ea5eb9a077f9
Reviewed-on: https://gerrit.libreoffice.org/8722Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

OutputDevice::EnableRTL() is a bit of a mess. It uses a runtime
variable to see if it is using a VirtualDevice, and it uses a
dynamic_cast to see if the object is a Window or a Control!

I have made it virtual and moved the knowledge of class specific
functionality from OutputDevice to VirtualDevice, Window and Control
as needed. OutputDevice::EnableRTL() functionality is then called.

Also: small formatting change to outdev.hxx, also included a note
that WindowImpl is a pimpl in window.hxx.

Change-Id: I44b66601c4457fb2e0bbc1014fb7acf8f6942f80
Reviewed-on: https://gerrit.libreoffice.org/8721Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

The vertical space of the options dialog is to small for the new
security page in German translation. The last entry is not visible
and the button could not be reached.
Solution: I added more vertical space to ROW_3 which is the postion
of the separation line. I added the hight of a button and of the line.
This fixes the issue with missing space for the last entry in section
especially for the German translation.

Change-Id: Icacfd4587d605e56438f3d7207ce5d257f313072
Reviewed-on: https://gerrit.libreoffice.org/8735Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

Change-Id: If269f9058bba37eca8abf4745fd30214fbdd1890

I occassionally get lockups in IceProcessMessages() called from QtCore,
I'm actually not exactly sure why, as theoretically two connections
from one app shouldn't be a problem, but since LO does its own
session handling, there's no need to the KDE/Qt code to be involved,
so prevent it from connecting to the session manager altogether.

Change-Id: Iebe20d4cb5403e5fea8bd5d8c1f69b62d1c2907b

Change-Id: I14e1f7ef217eb5e8e9db9f8962af868ab0a4ab81

commented out since it came in with
83521984

With the new parameter you can now supress the dialog for document
compare in writer.

Change-Id: I984ee75552e5c006332331510df5d437b687903f

With the document from fdo#75622, this saves 3775 calls and leaves only 13.
e586fe45 removed avoiding the call
to LayoutTable(), which made loading slow. I checked that the doc from that
bugreport still works, so if very original code was correct in avoiding
the call sometimes, this should be ok too.

Change-Id: Ia80f974d4497e5cb04612331527eb87b579ddb76

Change-Id: If271f02d99625ffa6fd418cf83307add7c09d802

Change-Id: I1442d57eb2f80d98370549659eece5a2cd7db3c5

Change-Id: I96786bcab1205d212926af6c7c6afbf88bfe1453

The problem was that in
writerfilter::ooxml::OOXMLStreamImpl::lcl_getTarget(), we went over the
list of all hyperlinks for each request. Instead, let's do it once, and
in the remaining cases just look up the result from a map.

Numbers before on my machine (ms, load time):

2215, 2243, 2205

After:

1362, 1358, 1358

So that causes about 39% speedup for a non-debug build.

Change-Id: Ib4718abbe834c5ba49a85469148b656e3c808041

The problem is already fixed by
33c310b6 and
1a67b7cc .

This reverts commit 363119a9.

Change-Id: Ieac14da9f37849cd51d14ed0035b3808b8dd77c0

Change-Id: I937a85fa7bda7df13e79e704973c21bfc732ba66

i115391 corected from isLocked to IsPasteResize which was used in modifiers for CustomShape and TextShape
(cherry picked from commit 473a118a)

...the calls to getAccessibleChild(0) and getHeaderBar(BBYTE_COLUMNHEADERBAR)
ultimately all ended up at
AccessibleBrowseBox::implGetHeaderBar(BBTYPE_COLUMNHEADERBAR).

Change-Id: Ie25659e19d0d1ce2dcde2d1440ecc69b4fa265b0

Change-Id: Ia9035189bd01d3a3e5e3222760fc392f3f78880f

Change-Id: Id522113f1ba0c9e7b1356acc01a79e0e90a5e752

Change-Id: I59fb7843d5c9a6cf2873b6d668d0e9dccff316d2

Change-Id: I9d85108b8c2659f1b167299840772f51f80b2e29

Change-Id: I6d8e1e3c78675b6096f354cace3ff060801cad9e

The isEmpty could be redundant, since the first test for length in
the old code seemed to be just a pre-condition for evaluating the
expressions in the second one.

Change-Id: I0d1838d5eb143196a0d7be0cbadf9f0a59cff71b

Change-Id: I5164e7d82f882ea9fe296a5a2df616d396d6726f

The code reads better this way, and certainly is easier on human brain
to process their logic.

Change-Id: I3150fca3385931b9fe01ad13cb11fab410331349

Change-Id: Id915a41e3cc50720d36955da9aa2c7efa6c9034c

Change-Id: I54e3b086c57b9b727ffdd5d3aa0c38c2c2ea2954

Change-Id: I60a552f14c4c4735f702581be18231e658cb30e7

Change-Id: I58e63b11c684d686e0e0830bb4e0e3b7fc9bbc32

Change-Id: I328eb08b719b5fdd51e1ca7c5c6fa3546ac90adb

Change-Id: I2ab4bb2f57c95d2d4b1029fd6e324ebbe0ff7148

Change-Id: Ia305ad3b1e04129e172f0f2fe3428f84e4654576

Change-Id: If80b842a79d476e5e90eeedc4520fc952c8f8f2a

Change-Id: I9f92a57644239b11941236a1cee5302eca993a7b

This code has been there at least since 2000-09-18, so removing it
seems to be safer than adding the evaluated token to the sMsg
OUString, as the surrounding code would suggest.

Change-Id: Ie9b2446b80a7bf79efbf6963b3a5e40548d4154c