Bug 136245 - Page count is terribly off on file open with a large table; updates after a few seconds (Windows Only)
Summary: Page count is terribly off on file open with a large table; updates after a f...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.3.0.4 release
Hardware: All Windows (All)
: low trivial
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Fields-Page-Count
  Show dependency treegraph
 
Reported: 2020-08-28 21:19 UTC by Telesto
Modified: 2024-08-05 10:48 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2020-08-28 21:19:00 UTC
Description:
Page count is terribly off on file open with a large table; ultimately it will work (but seems slower)

Steps to Reproduce:
1. Open attachment 107266 [details] (bug 84635)
2. Wait until the document appears
3. look at the page count - 43 
4. Wait rather long time -> Drops to 26 (expected)


Actual Results:
Opens with 43 pages

Expected Results:
26 pages straight forward, not first 43 and trimming down to 26


Reproducible: Always


User Profile Reset: No



Additional Info:
Found in
Version: 7.1.0.0.alpha0+ (x64)
Build ID: 6640d7f405d2970ba2825a9455926cc803284d01
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

and in
6.2

also in (opening fast, but number of spurious pages on file open (not being trimmed down). Exactly: 43 pages
6.1 

fine with 
6.0
Comment 1 Telesto 2020-08-29 20:44:49 UTC
My bibisect attempt lands here. 
author	Jan-Marek Glogowski <glogow@fbihome.de>	2018-07-09 09:06:55 +0200
committer	Khaled Hosny <khaledhosny@eglug.org>	2018-07-09 22:11:35 +0200
commit fad862e290d727fc9fefe206f6e4b807482c4175 (patch)
tree 5195c1277fce198e5b69d552e9209665d8007faf
parent 9d754a59154c40235c240bb0e7f47a2006fa85bd (diff)
tdf#118555 fix HFONT fallback handing / lifecycle
Instead of storing the never changing DC in the WinFontInstance
store the HFONT, which is Windows logical font instance.

Then set the correct HFONT instance from the layout when rendering
its text.

This also changes the HFONT ownership and lifecycle. The HFONT
is moved from the mhFonts to the WinFontInstance, if available,
so it has a proper referenced lifecycle. The mhFonts is still
needed, as embedded font just supply an HFONT and no
WinFontInstance.

https://cgit.freedesktop.org/libreoffice/core/commit/?id=fad862e290d727fc9fefe206f6e4b807482c4175

However coming without warranties, as there is also sorts of quirky behavior in the 6.2 bibisect repro.
Note 2: The 43 to 26 is also in 6.1, but is simply fast disappearing. So maybe i'm looking in the wrong bibisect repro?
Comment 2 Telesto 2020-08-29 20:47:30 UTC
@Buovjaga,
4 eyes so more than 2.. Any idea how to approach this?
Comment 3 Buovjaga 2020-08-30 12:20:06 UTC
(In reply to Telesto from comment #0)
> Description:
> Page count is terribly off on file open with a large table; ultimately it
> will work (but seems slower)
> 
> Steps to Reproduce:
> 1. Open attachment 107266 [details] (bug 84635)
> 2. Wait until the document appears
> 3. look at the page count - 43 
> 4. Wait rather long time -> Drops to 26 (expected)

Goes from 41 to 26, took maybe 10 secs.

Version: 7.1.0.0.alpha0+ (x64)
Build ID: a486fd929d4b3e915f928ef495b6cb2b96d74a3a
CPU threads: 4; OS: Windows 10.0 Build 19041; UI render: default; VCL: win
Locale: fi-FI (fi_FI); UI: en-US
Calc: threaded
Comment 4 QA Administrators 2024-01-25 03:26:04 UTC Comment hidden (obsolete)
Comment 5 wjsim 2024-03-07 22:21:28 UTC
This bug is still present in

Version: 24.2.1.2 (X86_64) / LibreOffice Community
Build ID: db4def46b0453cc22e2d0305797cf981b68ef5ac
CPU threads: 16; OS: Windows 10.0 Build 19045; UI render: Skia/Vulkan; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: CL threaded

The bug is also still present in 

Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 6a064b1967e06e40be40817deff99d00c1a8554f
CPU threads: 16; OS: Windows 10.0 Build 19045; UI render: Skia/Vulkan; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: CL threaded

Goes from 43 to 26, roughly about 2 to 3 seconds for me for both versions.
Comment 6 Buovjaga 2024-08-05 10:48:58 UTC
The same behaviour is seen already in the oldest commit of win32-4.3. So the timing of when it changes the count has just varied over the years.

The latest master opens the file much faster than previous versions and it only takes a few secs for the correct count to appear. I would not call a few secs a "rather long time".