New branch 'private/Ashod/cd-5.3-3.2_import_unloaded_share_GfxLink' available with the following commits: commit 13577d3980c0697d157125361bcf4e2dbe4dede9 Author: Ashod Nakashian <ashod.nakash...@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.nakash...@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.nakash...@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.nakash...@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.vajng...@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 <c...@libreoffice.org> Reviewed-by: Tomaž Vajngerl <qui...@gmail.com> (cherry picked from commit 7b355669c6ddeab2e6cec692d6afdff41c61d0fb) Change-Id: Ia1218f93b1735402b7828404f65660e2d4acf32f commit b2b020922bd601196cadca4baa7fa2ec1d734b41 Author: Miklos Vajna <vmik...@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 <vmik...@collabora.co.uk> Tested-by: Jenkins <c...@libreoffice.org> (cherry picked from commit e8a05109d91bb9e82fcec5204514766f4bdbbee8) Change-Id: Iee742a2cd3c581aeaf1a1ed9f55cd543955a85e0 commit 84fcdcec0f95487bb8b0eeafa79de8b389fccfdd Author: Noel Grandin <noel.gran...@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.gran...@collabora.co.uk> Tested-by: Noel Grandin <noel.gran...@collabora.co.uk> (cherry picked from commit ce301dadcbbf8d0f0a71c7eaebca9bfad2e08801) commit 636f44b146784a98f6d069b0f83bf7f765f13659 Author: Tomaž Vajngerl <tomaz.vajng...@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 <c...@libreoffice.org> Reviewed-by: Tomaž Vajngerl <qui...@gmail.com> (cherry picked from commit a2b53fece14f745853bcfe1a300c3dceb580e148) Change-Id: I5dbf74db4a6455d32c4abcbad7be21c7f0534642 commit 80d57f9e9277f2621cea6d810baff227a19cedba Author: Tomaž Vajngerl <tomaz.vajng...@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 <c...@libreoffice.org> Reviewed-by: Tomaž Vajngerl <qui...@gmail.com> (cherry picked from commit 7fbb96c2fbf99c47eca6f493f54fc3974f55cf86) Change-Id: I54a2fa6c7e997469aaa7770db05244adb9f64137 commit a41f48f4bcd5e833ded74e79edd55e942c6ee558 Author: Jens Carl <j.car...@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 <c...@libreoffice.org> Reviewed-by: Jens Carl <j.car...@gmx.de> (cherry picked from commit d31fdb27583f5f594e8424bf061a74fde5cf2704) Change-Id: Ibdae392533605b56cf9b9c28f8c419ea0d32c0d8 commit cda0cc8ab05718fcca75a9e18da14ce5249c8c2b Author: Miklos Vajna <vmik...@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 <c...@libreoffice.org> Reviewed-by: Miklos Vajna <vmik...@collabora.co.uk> (cherry picked from commit 252e524094893063e4bcc822571a14c4558e1d00) Change-Id: Icd7bd447e7ae623b0a8548e020d8f6ab38da47bb _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits