Change-Id: I934ad7453f35909f4c3ad999e33453b5b6032480
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Now text looks better, for instance the lower-case "t" glyphs on the
Start Centre aren't totally weird any more. But for instance the tip
of the hook of "j" leaks into the "i" texture. I guess I really would
need to render glyphs one by one.

Change-Id: I69ae2d2f7c559530bcfdfc1a4915503fcb3ab4af
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: I0bbea4ef62c8e94d8b8f1bfb440712da5839e532
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Shows text roughly correctly, but many problems. Conditional at
run-time on the SAL_ENABLE_GLYPH_CACHING environment variable in
addition to SAL_FORCEGL.

One obvious thing that needs fixing is making sure that the glyphs are
rendered into the texture without overlap. Compensating negative
values in the ABC::abcA and ABC::abdC width is not enough.

Also, the cache should be changed to actually have "cache"
functionality, i.e. entries should expire based on some criterion.

Change-Id: I66a37d3354a09011a654f15a7d2bd8efaa14ad1e
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Is comphelper the right place for this? Is having it as "inline" the right
way?

Change-Id: I973dbde108f89b6cab17e5d88db2390d6f18a672
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: Ia41882a4d33e7b148044801902517b2b034d3ee4
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Use Uniscribe also for non-complex text. It is complicated enough to
have separate Graphite and Uniscribe layout engines. Will make further
changes to the code easier to manage, especially as with the
UniscribeLayout code we have access to the actual
glyphs. (Cf. 3e47219e)

Change-Id: I9c67c172fe3e3d26d1c6cb1c0b7f1516b0b87f12
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Remove dead code. Should have no effect on behaviour.

Possibly originally the intent was that mbDisableGlyphs would have
been false in most cases on NT-based Windows (all versions that we
support now). However, since dadfc608,
in 2005, mbDisableGlyphs was set to always true in the SimpleWinLayout
ctor.

Change-Id: Id929224d5656706762c2f44ee26c76f8b20ee8b8
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

It is really ugly to use SAL_OVERRIDE inconsistently.

Change-Id: I8b556a9cc65e6f71198d126d07ce1559216543e9
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: I0d0cb1ef1e7b7f4747204b84c7c910f174e9c7b5
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>

Change-Id: I2b6ded138b9101415fc49e93e1ec3ebcd3a9d2ae
(cherry picked from commit 5ed690a3)
Reviewed-on: https://gerrit.libreoffice.org/18099Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Change-Id: I358758999bb918e73cdee2224e575e72c2131453
(cherry picked from commit 0c713e45)
Reviewed-on: https://gerrit.libreoffice.org/18098Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Change-Id: I60d3388ece28a69c31a85b9e3b495cbe8a03e7dc
(cherry picked from commit 42732d25)
Reviewed-on: https://gerrit.libreoffice.org/18097Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Change-Id: Ia593dd0e2239a97f17bb03f005d22028da482445
(cherry picked from commit d400f155)
Reviewed-on: https://gerrit.libreoffice.org/18096Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

moz323894-1.tiff and moz456356-1.tiff take too long to load

Change-Id: Iaafa064fd05e4a4152004e7ceb6256af68aeef01
(cherry picked from commit 7e373e92)
Reviewed-on: https://gerrit.libreoffice.org/18095Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

the paste special listener listens to owner changed, but expects
the owner changed event to contain the new contents, while the
original code expected getContents to be explicitly called to get
the new contents. So on owner changed always return the up to
date contents

Change-Id: Iaa7df2c01d360c3c5831b2258b4c2d41740fdffc
(cherry picked from commit 26786ae2)
Reviewed-on: https://gerrit.libreoffice.org/18054Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

As agreed in the bug report, a preview shouldn't be
themed by the OS theme.

(cherry picked from commit fa1807ad)

Conflicts:
	include/svx/numvset.hxx
	svx/source/dialog/svxbmpnumvalueset.cxx

Change-Id: Icdc5021c3ac614fe1d490fd513a407a81dbc169e
Reviewed-on: https://gerrit.libreoffice.org/18068Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>

... in a style with family "paragraph", as detected by a
--with-export-validation build due to the unit test added yesterday.

So use the loext namespace for the style:graphic-properties element,
and also omit it entirely (and rely on the style:paragraph-properties
attributes fo:background-*) if the ODF version is set to 1.2 or 1.1.

Also adapt a previous check that was missing the "drawing-page" style
family, where the style:graphic-properties is allowed (commit
9746dc9a)

(regression from 7d9bb549)

Change-Id: Iedd66483f63020328bd61e1c1e19c62787b8ff6b
(cherry picked from commit b1922eec)
Reviewed-on: https://gerrit.libreoffice.org/18064Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

Just invert the scrollbar the same way gtk+ does by checking the
button positions and adjust the drawing style accordingly.

Change-Id: I3475c33e074c1509a73da618a4efa44dc61a1a77
(cherry picked from commit 273df176)
Reviewed-on: https://gerrit.libreoffice.org/18033Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>

Change-Id: Id4419b6b902b742117e248870b6a663c9c58955f
(cherry picked from commit c249f93d)
Reviewed-on: https://gerrit.libreoffice.org/18052Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: David Tardon <dtardon@redhat.com>

Silence the glibc compile warning and actually handle a wrong
return value.

Change-Id: Icdbc8e655d4b8ad20699dfd0abdbf3405a49f898
(cherry picked from commit e4a3bf06)
Reviewed-on: https://gerrit.libreoffice.org/18034Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

Change-Id: I50e07779a541bbea6b285a855fb54d419d4aed13
(cherry picked from commit 52a52a21)

This reverts commit 350b00af.

Breaks CppunitTest_sc_subsequent_filters_test

Change-Id: I92963fa16aacbf01f4d6660eff7497347538371d
Reviewed-on: https://gerrit.libreoffice.org/18029Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>

Change-Id: I9c384587c127ca077803bf8aae63d0e54eda6aaf
(cherry picked from commit be95c743)
Reviewed-on: https://gerrit.libreoffice.org/18025Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Change-Id: Ided7f9376f41ee8cb1f6903e54a2d51e0e07e1a7
(cherry picked from commit a8b2dc80)
Reviewed-on: https://gerrit.libreoffice.org/18024Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Change-Id: I288f7ff0327831603eda6e827c8acbae678dfaff
(cherry picked from commit cadac840)
Reviewed-on: https://gerrit.libreoffice.org/18017Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Change-Id: I544a67e3706c7d12414cc075118ef2f0f5ddd0f6
(cherry picked from commit 334dba62)
Reviewed-on: https://gerrit.libreoffice.org/18015Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

just shove the unneeded pam into dead space

Change-Id: Id0a3d550cd9d54394888e6b2d62e21eaded24a7c
(cherry picked from commit d9edac5d)
Reviewed-on: https://gerrit.libreoffice.org/17998Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

so do that in the ctor and drop the arg from a bunch of places

(cherry picked from commit 3d992d77)

Change-Id: Iaad71e9854b38d8632df057dfe9c96468d1d0098
Reviewed-on: https://gerrit.libreoffice.org/17997Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Build ID will show enabled OpenGL with an extra string "-GL"
helping the fix of rendering issues.

Change-Id: Id7bf2db2edb165542bf7a2a253c698c494278a03
Reviewed-on: https://gerrit.libreoffice.org/18014Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

For example, "B2:" or ":B2" were accepted as valid range, with the
missing part set to A1, which resulted in a A1:B2 range.

An omitted sheet specifier always results in a valid relative sheet,
hence that flag is set, so testing flags for non-null before the final
result is available isn't sufficient.

Change-Id: I57d1b7cf71a298e7ac108b0e13aab88208f0f00c
(cherry picked from commit ed6209f5)
Reviewed-on: https://gerrit.libreoffice.org/18023Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

for the duration of inserting attributes, and excess
dos newline chars can be removed safely afterwards

Change-Id: If70e34fec1c0819f827f483d3d7ac4f19b3caef8
(cherry picked from commit cc596d8d)
Reviewed-on: https://gerrit.libreoffice.org/17983Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

(cherry picked from commit e3771338)

-Werror,-Wpessimizing-move

(cherry picked from commit 975a2bcc)

Change-Id: Ie95c36fe3705e2645a59cac117d99d7b85388ce1
Reviewed-on: https://gerrit.libreoffice.org/18013Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>

If using the same texture to store the image and mask data (for
example when using texture atlas) the mask and image (RGB) coords
aren't the same anymore. With this commit we always define the mask
coords separately.

(cherry picked from commit bdce4e29)

Conflicts:
	vcl/opengl/program.cxx

Change-Id: Ie33f87a6e9ab398972c6a3d5938e5f1364c82d36
Reviewed-on: https://gerrit.libreoffice.org/18012Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>

Plus two trivial follow-up build fixes.

Change-Id: I7fe90c0a0033b4d9a341a4f0b8356d7f7133e93e
(cherry picked from commit b26ba85d)
(cherry picked from commit 2792868d)
(cherry picked from commit 1e8e3a8c)
Reviewed-on: https://gerrit.libreoffice.org/18011Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>

... and disable/enable Apply/Ok buttons for in/valid ranges.

(cherry picked from commit ad1284df)

implement Edit modify handler, rhbz#1255811 related

Same as in ScRandomNumberGeneratorDialog.

(cherry picked from commit da9ecf1a)

implement Edit modify handler, rhbz#1255811 related

Same as in ScSamplingDialog.

(cherry picked from commit a88fecdc)

implement Edit modify handler, rhbz#1255811 related

Same as in ScStatisticsInputOutputDialog.

(cherry picked from commit b2363e98)

1bc9296bfc8b1b2b8f3fc20183e2c626f94dee09
ff709ec2284673efa8824a9b8cfc037e4d6d567a
0e3eb06bc86cf77c405c54f312340c7b2551c1ec

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

Change-Id: I2d1aa79a67bf130c3844b3bccb34a1761e8e4dfc
(cherry picked from commit 2ed0de57)

Change-Id: Ieedf4810f1e726bf11f0d0b1a203e560a0b16550

Change-Id: I86ae50bd8d2148e86a6c8fc0821d3e2bc29a0cd4
(cherry picked from commit e3cd97ce0b2e153b8b6812cc66f2e7d33504a7da)
Reviewed-on: https://gerrit.libreoffice.org/18006Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>

Change-Id: I23cbaf7fc1320168eb75ca33fd6d704125508785
(cherry picked from commit 666fb214)
Reviewed-on: https://gerrit.libreoffice.org/17982Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>