This reverts commit 7fa5f876,
which uses LIBO_INTERNAL_ONLY features in external code.

Enabled VCL Timer unittests on Windows builds and added two unittests to
detect misfiring of timer events.

Due to the nature of timer events having virtually no guarantees on
non-realtime operating systems, the tests may fail randomly under
different system loads.  The tests repeat a few times in such cases, but
still there are no guarantees.

These tests are added to detect gross breaks in the timer algorithms and
to improve certain corner cases.  They are designed to minimize any
random failures.

Posterity might hate me for committing such unittests, so I added
TEST_TIMERPRECISION directive to include/exclude the tests at compile
time. It should be enabled if it causes more harm than good.

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

Change-Id: I0db404b84f62db9636fa30f9af1b9cc0d70a2624

The button were not drawn at all

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

leave it untouched to make gtk3 rtl spin buttons possible
where the additional mirrorValue contains the region
of the spinbuttons to render within this rectangle

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

Change-Id: I797b986b227381bc066ea3aa305c0abf78cd156a

Change-Id: Ib48c6b05b7002db19e47d07fcfc0241468452190

Change-Id: I1b5125f5354dc2a1f4b4312dbe2b921d5cf13b49

...when border is around more than one paragraph

Regression from:
16f637da

Change-Id: Iea9455dd7cc3c301c3e2d5d7903c98d98e515595

This doesn't change the result of the BestIndex method

Change-Id: If1efc04feac3d4966c126b1fa93d017a4beafbe1

Change-Id: I2abd6aedb16303d8988c17e7204c9607bf8336a3

Change-Id: I635585f664e700047d92821fb965d42b803a8a15

Change-Id: I9c6f821f7bd1e9bd3eb8f47e269e62a792523f1a

Change-Id: I580c1c873b440d5230a0c9e6a2a2af5d5d8c0e0e

Change-Id: I3b4adb5ac75b9761ae058834e3e64779a07851c9

with the variadic variants.

Change-Id: I8a82df7ea24874d8e246dc983418d872f1943c8e
Reviewed-on: https://gerrit.libreoffice.org/17919Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

This addresses some cppcheck warnings.

Change-Id: Ia7f58c24429310b66f4464d8ba8b4f2e5c6fef38
Reviewed-on: https://gerrit.libreoffice.org/17922Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>

This addresses a cppcheck "variableScope" warning and adapts the
indentation of some code lines according to the surrounding code.

Change-Id: I76eca59a808740ab0366dd41c889a6ef19a2436d
Reviewed-on: https://gerrit.libreoffice.org/17923Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>

This addresses some of cppcheck's "redundantAssignment" warnings
("Variable '<varName>' is reassigned a value before the old one
has been used").

Change-Id: I8fd2950d5aa1a3f4bd9e1c4c336abe465d1f657e
Reviewed-on: https://gerrit.libreoffice.org/17926Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>

unnamed (None) Library/Executable will prevent correct
vs 2013 ide integration

Change-Id: Ia00c02609a83b5f891995e54fd3f365cd63cb7a4
Reviewed-on: https://gerrit.libreoffice.org/17933Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>

and some usage comments

Change-Id: I264d6e1ed52046e58284a2090eff3e8911c87f6a

See http://msdn.microsoft.com/library/cc233965.aspx

Change-Id: I5b86a309049806c2c67fe185acc421bf20de89cd

Change-Id: Id168238f85fcec6ead008e0757c584dfd4326883

If in some cases we don't want a visible separator, I can't
understand why it shouldn't apply also to platforms that
support native separator drawing.

Change-Id: Ib88bece62cfbb092808f3257a7ba9bd63f4cb1d7

Don't remove non-existent languages to clean the dialog.
Avoid crashing when adding languages to a non-localized dialog.

Change-Id: Ie356177e00e56c214727d513ae0ce6d62d32591c
Reviewed-on: https://gerrit.libreoffice.org/17929Tested-by: Jenkins <ci@libreoffice.org>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

ToggleFloatingMode can be run during dispose, and should handle that,
thanks to Maxim Monastirsky for the analysis.

Change-Id: I72c5aff9b75fa2c1b6795e239370244dafdafae2

http://nabble.documentfoundation.org/Bogus-ligatures-in-OpenSymbol-font-tp4157549.html

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

with the variadic variants.

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

Change-Id: Ic81e7a14701ff30aa46a9293dd6db351bd350ba5

This addresses some cppcheck warnings.

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

This addresses some cppcheck warnings.

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

Project: help  7a2cad733d9090afec58f6ac7cd9e97ca410a1fb

<item>Start </item><item>at</item> -> <item>Start at</item>

Change-Id: I45b5ab070d8b3ae2b76e9d32b73cdb9e1cb93551

Change-Id: Id5280f3ea136e061f4c1e21a60aeedf2d56d0bea

getSvxBrushItemFromSourceSet() is called once for each property of a
frame style that is set, and the "solid" fill-style is set by first
applying the BackColorRGB and then BackTransparency property.  So there
is an intermediate state that has fill-style NONE but a XFillColorItem
set - don't reset the color to white in that case!

Apparently writerfilter generally relies on the current reset-to-AUTO
implementation (loads of test failures), so for now limit the fix for
when we know that a frame style is imported from ODF.

(regression from 3d399b0e)

Change-Id: Ia987fe586ec819a1f3cd85acdbfb03c74ca0ec2c

Change-Id: If218ecd47435c261de14bd5f838c1e54f826a6ab

Change-Id: I7bc614dc45154c8dd5a7d3948f2129e5127876b3

As per [MS-LCID] rev. 7.0 2015-06-30

Change-Id: If4980d2b1149d93b39e5b57f5741bbdfb5680516

as ScTabViewShell::CreateRefDialog was still holding VclPtr
reference to them and never released it -- regression from
commit 087f7fe5 which is hereby
reverted.

Now it is *DlgWrapper holding the reference (via
SfxChildWindow::pWindow) and dialogs get properly destructed
when SfxChildWindow dies in SfxViewFrame::SetChildWindow(0, false)

Re tdf#91125, it turns out that the follow-up commit
06ecf7d2 is actually enough to
prevent the crash.

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

This addresses some cppcheck warnings.

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

In paint bucket mode the user can undo by right-clicking.
Undoing, however, is only possible after the first paint op.
After undoing is enabled, the user can undo indefinetely by
righ-clicking. This is not consistent and can be confusing.

This patch tracks the changes done after entering the paint
bucket mode, and allows the user to undo them using right-clicking
until they reach the first operation before entering this context.

Because the user can undo/redo using other means, the patch takes
it into account and will reset its tracking to accomodate this.

Ultimately, the user is able to make any changes in paint bucket
mode and undo them all by right-clicking, but no further than that.

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