[Libreoffice-commits] core.git: Changes to 'private/Ashod/cd-5.3-3.2_import_unloaded'
Ashod Nakashian
ashod.nakashian at collabora.co.uk
Mon Jun 18 11:55:21 UTC 2018
New branch 'private/Ashod/cd-5.3-3.2_import_unloaded' available with the following commits:
commit 13577d3980c0697d157125361bcf4e2dbe4dede9
Author: Ashod Nakashian <ashod.nakashian at collabora.co.uk>
Date: Sat Jun 9 15:31:42 2018 -0400
vcl: maintain shared PdfData across swap-in and -out
Change-Id: I6e434e5b5db0ddcd2fb4d1e522ffdb1c963c1f34
commit d271bb61e216556887dae60d7b7421c9cf236a1b
Author: Ashod Nakashian <ashod.nakashian at collabora.co.uk>
Date: Sat Jun 9 13:09:35 2018 -0400
vcl: sd: import PDF as unloaded images
Change-Id: Ibf9afeab2cb580a245af666b906310de1c9b351f
commit f8c68132d9f48b5e935a475c5f6de3aaf16da390
Author: Ashod Nakashian <ashod.nakashian at collabora.co.uk>
Date: Sat Jun 9 13:01:24 2018 -0400
vcl: load the correct page when swapping-in a PDF Graphic
Change-Id: I00ffb9f76e763e1c4cd65c5dcea061d6d94d1d6a
commit 0026d3c5e1ef966e5dbb6174537484104e85480c
Author: Ashod Nakashian <ashod.nakashian at collabora.co.uk>
Date: Fri Jun 8 22:54:48 2018 -0400
vcl: share GfxLink
When importing PDF as images, we store the
PDF stream in the GfxLink. For large PDFs
storing a copy of the full PDF with each
page is overkill. For example a 10MB PDF
with 200 pages will consume 2GB of memory!
Change-Id: I99913514cf5c562683080bc817668095bee69427
commit 1e47e0ab198b128c50c6bfec0b9486820ddd641e
Author: Tomaž Vajngerl <tomaz.vajngerl at collabora.co.uk>
Date: Sat Apr 14 15:13:05 2018 +0900
Function to load graphic swapped out (loaded on demand)
When a document is loaded it takes a lot of time and memory to
load the graphic that are in the documet, so avoid that and just
store the compressed graphic into a temporary file (handeled by
GfxLink) and load when we really need to show the graphic.
GraphicObject cached some attributes from Graphic, but this
attributes now aren't available immediately so this attributes
are removed form GraphicObject and now delegate to the Graphic
itself. GetSizeBytes attribute however was removed as it is
only used in some tests.
GfxLink initial values were moved to the constructor and are
not set in the header file anymore (as it is the recommended
way to do it).
The SdImportTest::testDocumentLayout failed as it looks like the
dump sometimes didn't include the width and height of the null
bitmap (which is set to 32x32) of the FillBitmap in some
situations, but then in other situations it did include this
attributes. With this change the width and height are always
included for the FillBitmap which looks like it is more correct.
Reviewed-on: https://gerrit.libreoffice.org/53016
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Tomaž Vajngerl <quikee at gmail.com>
(cherry picked from commit 7b355669c6ddeab2e6cec692d6afdff41c61d0fb)
Change-Id: Ia1218f93b1735402b7828404f65660e2d4acf32f
commit b2b020922bd601196cadca4baa7fa2ec1d734b41
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date: Mon May 8 16:52:00 2017 +0200
vcl: split jpeg import into two parts
Split the import into two:
1) Just create the bitmap, this part is not thread-safe (e.g.
OpenGLContext::makeCurrent() is called when OpenGL is enabled).
2) Import the image into an existing bitmap.
The point is that the second part takes much more time than the first,
and in the future that part may be executed on a thread, while without
such a split the whole ImportJPEG() can't do that. For now
GraphicFilter::ImportGraphic() simply invokes the two parts after each
other, so no real functional changes yet.
Reviewed-on: https://gerrit.libreoffice.org/37397
Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
Tested-by: Jenkins <ci at libreoffice.org>
(cherry picked from commit e8a05109d91bb9e82fcec5204514766f4bdbbee8)
Change-Id: Iee742a2cd3c581aeaf1a1ed9f55cd543955a85e0
commit 84fcdcec0f95487bb8b0eeafa79de8b389fccfdd
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date: Thu Sep 28 11:28:15 2017 +0200
convert GfxLink::IsEqual to operator==
exactly one use site
Change-Id: Id0a75d46c459be213e81b47aa6c1194c87aec848
Reviewed-on: https://gerrit.libreoffice.org/42891
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin at collabora.co.uk>
(cherry picked from commit ce301dadcbbf8d0f0a71c7eaebca9bfad2e08801)
commit 636f44b146784a98f6d069b0f83bf7f765f13659
Author: Tomaž Vajngerl <tomaz.vajngerl at collabora.co.uk>
Date: Wed Apr 4 17:51:26 2018 +0900
vcl: Add a internal (memory) manager for Graphic objects
This adds vcl::graphic::Manager which is a manager singleton that
tracks all the creation fo graphic objects and swaps them out on
a time and allocation basis.
Time based - every number of seconds it looks for Graphics that
weren't used for a time.
Allocation based - when creating a new Graphic and the total of
Graphic uses more than the total amount of memory for Graphics
defined in configuration, it tries to release the Graphics that
weren't used for a time.
Reviewed-on: https://gerrit.libreoffice.org/52396
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Tomaž Vajngerl <quikee at gmail.com>
(cherry picked from commit a2b53fece14f745853bcfe1a300c3dceb580e148)
Change-Id: I5dbf74db4a6455d32c4abcbad7be21c7f0534642
commit 80d57f9e9277f2621cea6d810baff227a19cedba
Author: Tomaž Vajngerl <tomaz.vajngerl at collabora.co.uk>
Date: Sun Mar 11 11:16:34 2018 +0900
Move (and rename) graphic stuff from svtools to vcl
This is preparing to change how GraphicManager works where it
won't base itself around GraphicObject anymore but Graphic. No
functional or cosmetic change was made to the classes, only
changes that were needed because of the move and rename.
The only thing that wasn't moved is the GraphicRenderer as it
is not needed in vcl for now (but makes sense to move it in the
future to keep graphic stuff together).
grfmgr was renamed to GraphicObject as the GraphicManager will be
changed a lot and most likely moved out, so the name grfmgr won't
make any sense anymore.
All the UNO implementations were renamed with a prefix Uno and
used the same name as the class name. This is made to be more
specific which are the Uno objects (for example graphic.cxx
contained the implementation of XGraphic, which is similar to
graph.cxx contains Graphic).
Reviewed-on: https://gerrit.libreoffice.org/51068
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Tomaž Vajngerl <quikee at gmail.com>
(cherry picked from commit 7fbb96c2fbf99c47eca6f493f54fc3974f55cf86)
Change-Id: I54a2fa6c7e997469aaa7770db05244adb9f64137
commit a41f48f4bcd5e833ded74e79edd55e942c6ee558
Author: Jens Carl <j.carl43 at gmx.de>
Date: Thu Nov 2 00:19:40 2017 +0000
Remove obsolete sc_condformats unit test
Remove the special snowflake sc_condformats because it's
now covert by xsheetconditionalentries (see commit
44e9640748791f602edb22cbc499200283466e1e).
Reviewed-on: https://gerrit.libreoffice.org/44523
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Jens Carl <j.carl43 at gmx.de>
(cherry picked from commit d31fdb27583f5f594e8424bf061a74fde5cf2704)
Change-Id: Ibdae392533605b56cf9b9c28f8c419ea0d32c0d8
commit cda0cc8ab05718fcca75a9e18da14ce5249c8c2b
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date: Tue May 23 12:10:44 2017 +0200
svtools: add GraphicProvider::queryGraphics()
This allows moving the for() loop from oox to svtools when importing
multiple images.
That means in case later we parallelize that loop, then the performance
benefit won't be restricted to oox, but also will be available for all
clients of the graphic provider.
Reviewed-on: https://gerrit.libreoffice.org/37945
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
(cherry picked from commit 252e524094893063e4bcc822571a14c4558e1d00)
Change-Id: Icd7bd447e7ae623b0a8548e020d8f6ab38da47bb
commit e9f049a5c6721fd29599292de1d00be21a07a5e5
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date: Tue May 23 10:20:03 2017 +0200
oox: add GraphicHelper::importGraphics()
Similar to GraphicHelper::importGraphic(), but can import multiple
streams with one function call.
Reviewed-on: https://gerrit.libreoffice.org/37942
Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
Tested-by: Jenkins <ci at libreoffice.org>
(cherry picked from commit b1319842a49cdf6512bbd9e81081e2a9edbd6089)
Change-Id: I5fd398bb6649259e86967f8db5cc1e212f50bc8e
commit 934bd8100e1a7df4104c065e2d69474d75acfa6a
Author: Tomaž Vajngerl <tomaz.vajngerl at collabora.co.uk>
Date: Wed Mar 7 16:31:16 2018 +0900
transformer: clean-up and pass originURL to transfored graphic
Change-Id: Iab01a5123ce6774120e2a0242616511a9fbe4403
Reviewed-on: https://gerrit.libreoffice.org/50867
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Tomaž Vajngerl <quikee at gmail.com>
(cherry picked from commit 19b6b8f8946a6fb720806a3f51a06760047d2ffc)
commit f8913533997d1c869e17325c4484f77a86b342b7
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date: Wed Feb 7 15:09:12 2018 +0200
move setAlphaFrom from GraphicTransformer to BitmapEx
Reviewed-on: https://gerrit.libreoffice.org/49360
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin at collabora.co.uk>
(cherry picked from commit 5531e6338bc9d7d5676a5c52059d5ccb2fcccb0c)
Change-Id: I69329c61e81791db3806fabe3256571fa937deb6
commit da91ffc94ac6772e593e13257896fec52e311eb9
Author: Chris Sherlock <chris.sherlock79 at gmail.com>
Date: Sun Jan 21 04:48:50 2018 +1100
vcl: remove BitmapColor Color() operator
BitmapColor has a Color() operator. However, this is confusing and
tends to hide that the two classes aren't the same. I have converted this
to GetColor().
Disable some tests that are failing to build due to missing << operator.
Reviewed-on: https://gerrit.libreoffice.org/48245
Reviewed-by: Stephan Bergmann <sbergman at redhat.com>
Tested-by: Stephan Bergmann <sbergman at redhat.com>
(cherry picked from commit 01e470f0ab49e458c57b0f28636a517b2bfb1b4a)
Change-Id: I0be2dcb3fc420e7be9c8d04330e7a3fe69a5412a
commit dcae336143f25a2fb98d4c4adee6df4e5cef4053
Author: Tomaž Vajngerl <tomaz.vajngerl at collabora.co.uk>
Date: Mon Apr 2 18:38:39 2018 +0900
graphic: rename Set{Get,Is}Link to Set{Get,Is}GfxLink
The main reason is to differentiate when we have the GfxLink from
other (url, file) links which use the same terminology.
Reviewed-on: https://gerrit.libreoffice.org/52245
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Tomaž Vajngerl <quikee at gmail.com>
(cherry picked from commit 11163faafe718e65b25f0dfea8700d1a18d4dee6)
Change-Id: I805c61e7fb231062340562c51c25a189b2cfc7f7
commit c5a524588fc4ec44fc0008517751bb1041b25deb
Author: Ashod Nakashian <ashod.nakashian at collabora.co.uk>
Date: Sat Jun 2 23:57:04 2018 -0400
vcl: refresh font cache only on real change
GTK3 emits way too many style-updated, which triggers font cache updates. The
avoidance of unecessary font cache updates was first implemented in 29c55564.
Unfortuantely, it seems that the current font-options was never set as
last-seen font-options, so still to many fonts-changed were emitted
Change-Id: I01a47d4d7fb033e335b1a49ffa6e1bb98f7fd28d
Reviewed-on: https://gerrit.libreoffice.org/55287
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
Tested-by: Caolán McNamara <caolanm at redhat.com>
(cherry picked from commit 08814a34e3a3841c316dfb27a07583049abc5195)
More information about the Libreoffice-commits
mailing list