Commit Graph

398872 Commits

Author SHA1 Message Date
Khaled Hosny
3c80eea38a Avoid potential HarfBuzz asserts
We need to update the buffer content type ourselves since we are now
using the low lever shape plan API that does not do this.

Change-Id: I043d15731cf206b142c3153896e194a822a70ffb
2016-10-30 18:29:30 +02:00
Zdeněk Crhonek
425c92291b add LEN test case
Change-Id: I99ee54406f446d9b97a61cbd3ec1a58115b544bf
Reviewed-on: https://gerrit.libreoffice.org/30392
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-10-30 16:17:04 +00:00
Zdeněk Crhonek
20dd5fbecc add FORECAST.ETS.ADD test case
Change-Id: I841115854791b329417b7ff0798e55dfccf93a9e
Reviewed-on: https://gerrit.libreoffice.org/30357
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-10-30 16:16:13 +00:00
Maxim Monastirsky
66b75b4ba9 tdf#103087 Border widget must not have focus initially
... unless it was opened using the keyboard.

Change-Id: I04675211ff043acf7757afed7252e7d7493eadad
2016-10-30 16:55:01 +02:00
Maxim Monastirsky
94876fe270 Let Menu dispose submenus
(I'm not sure about how good are the changes from ScopedVclPtr
to non-scoped, and disposeAndClear to clear. They aren't really
needed, because of the VclReferenceBase::mbDisposed logic. But
at least they should be safe, as long as we have disposeOnce
calls in Menu's dtor.)

See also previous commits:

4433d95b37
("MenuItemData now properly disposes the submenu")

89c23b4aae
("Sub menus no longer need manual disposing")

Change-Id: I9d455a94590f5eec9b097947f6984f1b3e477b52
2016-10-30 15:50:31 +02:00
Maxim Monastirsky
99da628e17 BmkMenu is dead now
Change-Id: I9b6e762fd4c231a5bd54eb546026499da2efa0e6
2016-10-30 15:40:39 +02:00
Khaled Hosny
0fd8ef082d Rename ServerFont to FreetypeFont
This is what it is actually is, we dropped support for server-side fonts
for a long time now. Renamed also a few related classes, but left
ServerFontLayout* ones as they will go away soonish.

Change-Id: I68a6dad51b6972368b7bf85a0b9c8089cc12740e
Reviewed-on: https://gerrit.libreoffice.org/30390
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
Tested-by: Khaled Hosny <khaledhosny@eglug.org>
2016-10-30 12:54:00 +00:00
Khaled Hosny
be1750d866 ENABLE_GRAPHITE_DWRITE is always set to 1
We don’t have a configure option to turn it off and I don’t think anyone
tries building without it. Furthermore, we use the code guarded by this
even without Graphite.

Change-Id: I0fb434af58a51370a66b5ad7eb89b145e4c015de
Reviewed-on: https://gerrit.libreoffice.org/30389
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
Tested-by: Khaled Hosny <khaledhosny@eglug.org>
2016-10-30 01:53:33 +00:00
Stanislav Horacek
0a4e0dfffd Updated core
Project: help  14ae4df6fadec87540994bb8698b65d05fae4763

update description of File - Properties - General tab

Change-Id: Ia172fb015d73dd7945918fc9a7ef9c3c89539add
Reviewed-on: https://gerrit.libreoffice.org/30383
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2016-10-30 00:15:45 +00:00
Zdeněk Crhonek
40c4cc9a6b add MAXA testcase
Change-Id: I05a15da6526449342c9f0f41960c461e67df50ec
Reviewed-on: https://gerrit.libreoffice.org/30353
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-10-29 23:08:55 +00:00
Khaled Hosny
7b4d921d8e tdf#103401: Fix applying DX array with font fallback
Turns out if we don’t adjust the glyph width (not only its X position)
we will apply the DX adjustment more than once when there is font
fallback.

Change-Id: I5e72e5d211adf3b378ec23c0cabe38fd0d5c9b19
2016-10-30 00:41:41 +02:00
Jochen Nitschke
b4cf309bab silence false positive warnings in non DBG_UTIL builds
Change-Id: I80428ebe570584fc7a31581207da0b27a16949af
Reviewed-on: https://gerrit.libreoffice.org/29244
Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
Tested-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
2016-10-29 21:51:01 +00:00
Tomaž Vajngerl
524dba884d icon theme name suffixing with "_svg" is unneeded
Change-Id: Ie0f9964d893c520ad02bb33a1d30821735392076
Reviewed-on: https://gerrit.libreoffice.org/30382
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-29 21:48:55 +00:00
Tomaž Vajngerl
b7c2764145 convert usages of DPIScalingFactor from int to float
Change-Id: I049b3d2cde4dcb8c8c0690d75a015a43cb71b0c0
Reviewed-on: https://gerrit.libreoffice.org/30381
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-29 21:48:30 +00:00
Tomaž Vajngerl
a0cf18f58f HiDPI: remove more unneeded scaling of icons
Change-Id: I521024af62ea15db80f5e851875b18dcf18556e4
Reviewed-on: https://gerrit.libreoffice.org/30380
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-29 21:48:09 +00:00
Tomaž Vajngerl
69b6ab1f8d tdf#100164 change scaling unit to precentage for *.5x factors
Currently we support DPI scaling by a integer factor. This commit
changes that to percentage so we can have scaling factors like
1.5x or 1.25x. This is useful with 2.7k monitors that are in
between standard DPI and HiDPI. Thresholding was adjusted to scale
to 1.5x when DPI is between 120 and 168 DPI.

The old method GetDPIScaleFactor has been changed to return a
float value insted of int. Sometimes it is however more accurate
to use GetDPIScalePercentage which was added in this commit.

Change-Id: Iaecee793ff3d5084d00adeebbcf5d7368c580882
Reviewed-on: https://gerrit.libreoffice.org/30379
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-29 21:47:41 +00:00
Tomaž Vajngerl
a86144696d surpress automatic scaling / conversion for document shadow
Change-Id: Ib020161db8d5e241e923dcd629a9a4e4bb4c23da
Reviewed-on: https://gerrit.libreoffice.org/30342
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-29 21:47:12 +00:00
Zdeněk Crhonek
6d68b2bbc9 add HYPGEOMDIST test case
Change-Id: I996161ff95fb682f3a9bb62fe6444be96fd6fd3f
Reviewed-on: https://gerrit.libreoffice.org/30355
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-10-29 18:54:57 +00:00
Zdeněk Crhonek
4d67dca78d add LOGINV test case
Change-Id: Ie05a462418322aac43f36c069749fdae195a8585
Reviewed-on: https://gerrit.libreoffice.org/30354
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-10-29 18:43:27 +00:00
Tomaž Vajngerl
4a4fd38c87 vcl: add bitmap::loadFromResource with additional flags
Sometimes we want to surpress automatic scaling and converting
for dark theme. This adds a loadFromResource function with
additional flags which makes this possible.

Change-Id: I893cee82ad6942644755f0c5b772fb5f073bbf00
Reviewed-on: https://gerrit.libreoffice.org/30341
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-29 18:15:04 +00:00
Tomaž Vajngerl
484390a096 HiDPI: remove unneeded image scaling when HiDPI is enabled
We don't need to scale images for HiDPI mode anymore - this is now
done automatically when the image is loaded.

Change-Id: Ie285cf4d672f4c098a468d3f7560ee2700365b92
Reviewed-on: https://gerrit.libreoffice.org/30340
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-29 18:14:23 +00:00
Tomaž Vajngerl
5bb5463efe tdf#51733 support SVG icon theme, disk cache and scaling / darken
Also resolves tdf#92248

This extends icon name resolving so that is in addition tries to
find an icon with the "svg" extension and load that instead of
stated (mostly "png") extension. If the filename extension is
"svg" we load the icon with the SVG filter instead.

This also adds icon scaling and conversion for HiDPI or when a
dark theme is wanted. If the SVG icon is available, we render it
at a higher resolution instead of scaling

As loading of SVG icons can be computatunally expensive, a icon
disk cache was added. This saves the rendered SVG as a PNG image
into the "cache" folder. The same caching is also used for HiDPI
and dark theme converted icons so we don't always scale or
convert the icons.

In addition some style changes and DRY fixes were made to the
ImplImageTree source code.

Change-Id: I9e421395a342ffe8da9facea7ea06e5db2778b26
Reviewed-on: https://gerrit.libreoffice.org/30339
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-29 18:14:01 +00:00
Eike Rathke
c857e5ef42 display matching search per selected category, tdf#67974 follow-up
Also select category All while typing in Search, and display the LRU list
independently.

Change-Id: I8a734cc202e017d3a10d8a6f89b1814073a77c81
2016-10-29 19:25:49 +02:00
Tamás Zolnai
24e522f723 tdf#103753: DOCX: Handle relative positions to left or right margin
Positioning relative to left or right margin in MS Word
works the same as the positioning relative to left or right
page border in LO Writer.

Change-Id: I476a5e9e76f766b7fb7f1c7f4a068af8bb3c8813
Reviewed-on: https://gerrit.libreoffice.org/30376
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2016-10-29 16:43:27 +00:00
Khaled Hosny
f351b3624d Add layout engine info to the About dialog
Change-Id: I6a87efa5e1072bac3588226df0e1351aa51c4d0d
Reviewed-on: https://gerrit.libreoffice.org/30108
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
Tested-by: Khaled Hosny <khaledhosny@eglug.org>
2016-10-29 16:34:31 +00:00
Mike Kaganski
1c708eee63 tdf#103473: Ensure positive rectangle size
Also remove conversion of both negative scales into rotation,
because it is handled by flip; use strict comparison instead of
approximate float less because it's correct here, and also because
basegfx::fTools::less ultimately uses rtl_math_approxEqual, which
description states: attention
approxEqual( value!=0.0, 0.0 ) _never_ yields true.

Unit test included.

Change-Id: Ia878fce360cf8aed539d95f2d4a1a3ad94379978
Reviewed-on: https://gerrit.libreoffice.org/30373
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2016-10-29 16:17:06 +00:00
Eike Rathke
8a0318e4f8 filter hidden functions also in search, tdf#67974 follow-up
Change-Id: Iaa23759e01d8f86bfc5c0bbd8b333cd331b7b8cd
2016-10-29 17:59:05 +02:00
Abhilash Singh
70e97ade2f tdf#67974 UI: Add a search box in function wizard in Spreadsheet
Change-Id: Idd243824a25ba99b2163f094c531dad7fe1153c8
Reviewed-on: https://gerrit.libreoffice.org/30118
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
2016-10-29 15:32:51 +00:00
Caolán McNamara
b696781f03 crashtesting: piles of pdf import crashes
e.g. ooo112863-2.pdf

Change-Id: I359f7e50f416f0a756e16099cdf47be3b1c32d90
2016-10-29 15:36:59 +01:00
Khaled Hosny
930a3d462b tdf#33278: Enable UI for vertical left-to-right
Used mainly for traditional Mongolian. This has been disable since 2010
(shortly after the code was merged):

commit faf68cc756
Author: Mathias Bauer <mba@openoffice.org>
Date:   Fri Jun 25 11:10:43 2010 +0200

    CWS mongolianlayout: disable new direction until problems are fixed

No idea what the problems are, but lets enable it and see what happens
(the actual layout code have been enabled all the time and one can
always manually craft ODT files that use it anyway).

Change-Id: Id38d1735013d451f9fe116876483df3484dec051
Reviewed-on: https://gerrit.libreoffice.org/30364
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
Tested-by: Khaled Hosny <khaledhosny@eglug.org>
2016-10-29 13:18:57 +00:00
Justin Luth
bb615bc146 tdf#41542 rename variable to match LO5.4 terminology
Changing /Allow_*Spacing_*Without_*Borders/I
       to AllowPaddingWithoutBorders

related to tdf#103275 where "spacing to contents" is to be changed to
Padding. Rename this already in LO53 to simplify potential backports
and laying other groundwork for fixing this bug.

Also, I can easily see setting AllowPaddingWithoutBorders globally in 5.3
for the purpose of being able to share documents when
the UI in LO5.4 permits creation of padding without borders.
Otherwise older versions will display significantly different formatting.

Change-Id: I253173274f824a019ebc09a039c471d170c1be73
Reviewed-on: https://gerrit.libreoffice.org/30372
Reviewed-by: Justin Luth <justin_luth@sil.org>
Tested-by: Justin Luth <justin_luth@sil.org>
2016-10-29 12:17:28 +00:00
Tomaž Vajngerl
f19f88a0d4 vcl: move method to the appropriate file, some C++11-ification
Change-Id: If51d16673c8b241487cae5305e293f213b7db5cb
Reviewed-on: https://gerrit.libreoffice.org/30338
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
2016-10-28 20:55:09 +00:00
Michael Stahl
87c518593d sc: remove antique reference counting hacks from ScDocumentPool
This attempt to prevent overflowing a 16-bit counter was obsoleted by
the conversion of SfxPoolItem's reference count to ULONG in 2001 or so.

Change-Id: Iafb6f151f68cbb84fda59bd134a7a4930f9a4d1f
2016-10-28 22:49:28 +02:00
Michael Stahl
85c38f1cf4 svl: change SfxPoolItem ref-count to sal_uInt32
Fixes the inconsistency between potentially 64-bit sal_uLong and
the max-value macros that are ~2^32.

Change-Id: I895c674819cf4766cb2c7441f670bc1305362a74
2016-10-28 22:49:28 +02:00
Michael Stahl
0117a5d34e svl: more SfxPoolItem asserts
Change-Id: Ic73fe09fc401359f043269fc8e5a1910fc8ddb02
2016-10-28 22:49:28 +02:00
Michael Stahl
855e69d37e svl: SfxPoolItem reference counting assertions
Change-Id: Ice2ec9a4592a1fad36913ae7d089aa8c63dfc669
2016-10-28 22:49:28 +02:00
Khaled Hosny
3d456dfa66 tdf#98879: Fix vertical text on Windows for the new layout
The DirectWrite rendering code wasn’t handling vertical text correctly
at least when used with CommonSalLayout. It has been fixed and now made
the default.

We are also doing all vertical text handling on our own, so loading
special vertical fonts on Windows is now disabled.

Change-Id: I10d150367fe59c77459bb1b813b6f31a0ff0c5d0
Reviewed-on: https://gerrit.libreoffice.org/30346
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
Tested-by: Khaled Hosny <khaledhosny@eglug.org>
2016-10-28 19:22:54 +00:00
Justin Luth
c0baab9aa8 reset border should clear all borders
If no borders were originally defined, the reset button was failing
to clear the last-selected border line.

Eg. page border, click on two of the borders, then hit reset. One border
remains defined, and didn't reset to the starting value of non-visible.
Because one border was still "selected", it became visible again as
SelStyleHdl_Impl reinitialized.

Change-Id: I2d20ae1e02816d5b30fe126572b54e7083114fc3
Reviewed-on: https://gerrit.libreoffice.org/30365
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Justin Luth <justin_luth@sil.org>
2016-10-28 18:58:49 +00:00
Stephan Bergmann
25df5f2ac7 loplugin:unnecessaryoverride
Change-Id: I70ba8d3b03c56a597785f109f0cb687bd936bc64
2016-10-28 19:48:44 +02:00
Miklos Vajna
5c33253daf xmlsecurity PDF verify: start using offsets from xref streams
This is needed (but not enough) to verify PDF 1.5 signatures. What's
missing next is support for object streams.

Change-Id: I5afec0a77839ffabe0aaa07e367064210535a1a9
2016-10-28 18:34:14 +02:00
Miklos Vajna
56cc352d82 xmlsecurity PDF verify: initial support of cross-reference streams
This adds support for cross-reference streams (which can be used instead
of plain-text cross-reference tables) + also one stream predictor.

The actual parsed data is still not used, though.

Change-Id: Ia806abd8a97636a1bd25dfdafea377b088800f00
2016-10-28 18:20:20 +02:00
Justin Luth
b41e7b817e tdf#63846 assign macros after VBA project fully loaded.
fixes regression caused by d4743045a0.

see the documentation for VbaMacroAttacherBase, which indicates that
the macros are to be attached after all objects are finished loading.

Made attachMacros() public so it could be called separately since so
many things are inter-dependent in the logic flow.

Change-Id: I11f7f4a1a75d6f77a97e12e3359d6ea8a995f518
Reviewed-on: https://gerrit.libreoffice.org/30337
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-10-28 16:12:27 +00:00
Zdeněk Crhonek
eef81dd68f add MAX test case
Change-Id: Ie32504b84e227cc3f33d89ec46bca1835e433844
Reviewed-on: https://gerrit.libreoffice.org/30352
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-10-28 16:08:59 +00:00
Stephan Bergmann
1ec08fe00b loplugin:unnecessaryoverride
Change-Id: Ib75f2c8c06c6219aa0f2a8b2b29d19fb95e09527
2016-10-28 18:02:32 +02:00
Tamás Zolnai
b927c1f4b3 tdf#103544: DOCX exp.: Image loss when have a frame anchored to the same para.
Regression from:
83d51e5e52

Partly revert the commit causes this regression.
I checked the related bugs (tdf#78590,tdf#80748)
intended to be fixed by this commit and reverting
this part does not bring back the corruption.
I guess something changed in frames' and text boxes'
import in the meantime, because this MergeMarks::IGNORE
is useless now.

Change-Id: If17776e8628561961c7ce2a2994e3fc609f75639
Reviewed-on: https://gerrit.libreoffice.org/30351
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2016-10-28 14:10:41 +00:00
Noel Grandin
8a22bc93e0 update unnecessaryoverride plugin to find pure forwarding methods
which can be replaced with using declarations.

Is there a more efficient way to code the search? Seems to slow the
build down a little.

Change-Id: I08cda21fa70dce6572e1acc71bf5e6df36bb951f
Reviewed-on: https://gerrit.libreoffice.org/30157
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-10-28 12:56:17 +00:00
Rohan Kumar
99fbcffa3d tdf#89329: use unique_ptr for pImpl in fpicker/
Change-Id: I4a5db5ceafe509bffc32a8800ab3d678f41244f7
Reviewed-on: https://gerrit.libreoffice.org/30309
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-10-28 12:26:09 +00:00
Caolán McNamara
03c7cfb6a6 note yet another color selector
Change-Id: I781da1ec218148dcb723db04dfe6ef3eacffe293
2016-10-28 13:15:34 +01:00
Caolán McNamara
d3e33f7ba8 ImplInitMenuButtonData can be folded into ctor now
because theres no ResId loader anymore

Change-Id: Id90786d710fe75e47c162fd00ad7fd672b9cee30
2016-10-28 13:03:45 +01:00
Caolán McNamara
624ff6633c only draw menubutton separator when two sides do something different
Change-Id: If6e3b72eacef2325f9f30770bbed32c627bc035c
2016-10-28 12:37:05 +01:00