Bug 113645 - Scrolling using the vertical scroll bar has become more choppy with OpenGL enabled
Summary: Scrolling using the vertical scroll bar has become more choppy with OpenGL en...
Status: RESOLVED WONTFIX
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
5.3.0.1 rc
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, perf
Depends on:
Blocks: Scrolling-PageUpDown
  Show dependency treegraph
 
Reported: 2017-11-04 13:18 UTC by Telesto
Modified: 2020-08-28 18:44 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
Bibisect log (2.90 KB, text/plain)
2017-11-04 13:19 UTC, Telesto
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2017-11-04 13:18:43 UTC
Description:
Scrolling using the vertical scroll bar has become more choppy with OpenGL enabled

Steps to Reproduce:
1. Open attachment 136841 [details]
2. Set view to multi-page view
3. Zoom out to 4 pages
4. Wait until all pages are loaded (drop in cpu usage)
4. Scroll down by selecting the vertical scroll (left mouse click) slider and move it up and down. 

Actual Results:  
Very choppy scrolling experience

Expected Results:
A bit more smoothed like before the identified commit. Without the commit is 'OK' but far from perfect compared to GDI


Reproducible: Always


User Profile Reset: No



Additional Info:
Found in
Version: 6.0.0.0.alpha1+
Build ID: 06cad1a9a42ea74434f9ed0e4027163d029eb4a1
CPU threads: 4; OS: Windows 6.3; UI render: GL; 
TinderBox: Win-x86@42, Branch:master, Time: 2017-11-02_23:40:06
Locale: nl-NL (nl_NL); Calc: CL

and in 
5.3.0.1


User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0
Comment 1 Telesto 2017-11-04 13:19:27 UTC
Created attachment 137519 [details]
Bibisect log

Bibisected to:
author	Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk>	2016-05-31 03:26:04 (GMT)
committer	Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk>	2016-06-08 02:39:22 (GMT)
commit 18260a0d9a381a105ffb032ee8f1050c77062102 (patch)
tree 8ebdd55593e4d7efe718ee8a8282d37cc0776c4e
parent bb157523310eb237dc16818f24df58d110242367 (diff)
opengl: deferred texture drawing in RenderList, add drawAlphaRect
Drawing accumulated textures (in Accumulatedtextures) is independent
of drawing with render list which causes problems with rendering
order when render list and accumulated textures are flushed. To
solve this we need to combine both so we can check for overlapped
drawing.

Previously drawRect was using RenderList batch drawing but not
drawAlphaRect which is essentially the same as drawRect but
additionally supports alpha value. This adds support to draw
alpha rectangles to RenderList and converts drawAlphaRect.
Comment 2 Xisco Faulí 2018-06-12 11:07:34 UTC
Hi Telesto,
is this issue still happening in master ?
Comment 3 Telesto 2018-06-12 20:33:12 UTC
Repro with
Version: 6.2.0.0.alpha0+
Build ID: 51aa57cd8ed46d28262e0d315328231f0fa814f4
CPU threads: 4; OS: Windows 6.3; UI render: GL; 
TinderBox: Win-x86@42, Branch:master, Time: 2018-06-12_05:54:56
Locale: nl-NL (nl_NL); Calc: CL

-> Only when using the vertical scroll slider with left click; fine when using the scroll wheel
Comment 4 Telesto 2019-01-16 10:03:33 UTC
Repro with
Version: 6.3.0.0.alpha0+
Build ID: 61778fd20395794d2de3b52d60dcc65083aecd93
CPU threads: 4; OS: Windows 6.3; UI render: GL; VCL: win; 
Locale: nl-NL (nl_NL); UI-Language: en-US
Calc: CL

Note: debug build...
Comment 5 Xisco Faulí 2019-07-31 11:59:20 UTC
Hi Telesto,
After all the recent OpenGL improvements, could you please try to reproduce this issue again in master?
Comment 6 Telesto 2019-10-20 16:22:21 UTC
Repro with
Version: 6.4.0.0.alpha0+ (x86)
Build ID: c45d477b0a0038d9c25176cf7cff299e5ddf3a7a
CPU threads: 4; OS: Windows 6.3; UI render: GL; VCL: win; 
TinderBox: Win-x86@42, Branch:master, Time: 2019-09-30_05:06:55
Locale: nl-NL (nl_NL); UI-Language: en-US
Calc: CL
Comment 7 Buovjaga 2020-08-28 13:11:50 UTC
As Skia with Vulkan will replace OpenGL UI rendering on all platforms, it does not make sense to keep OpenGL UI reports open.

Details about Skia: https://www.collaboraoffice.com/success-story/implementing-vulkan-capable-libreoffice-user-interface-using-the-skia-library/
Comment 8 Telesto 2020-08-28 18:44:16 UTC
Working OK for me
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