ODFstudio/odfstudio
2
0
Fork 0
Code Issues Packages Activity

tdf#157312 Don't change priority

Browse Source 
Instances of this class are constructed with
TaskPriority::POST_PAINT, but then it was set to
TaskPriority::HIGHEST when reused. Flushing
seems to be expensive (at least with Skia/Metal) so keep the
existing priority when reused.

Change-Id: Ieb40cd6122f543f76e66456cc92a8643e2320d73
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/157162
Tested-by: Jenkins
Reviewed-by: Patrick Luby <plubius@neooffice.org>
This commit is contained in:
Patrick Luby
2023-09-22 15:03:43 +02:00
parent bb3a60cfdb
commit f4c2c7c79c
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
vcl/skia/gdiimpl.cxx
+7 -1
View File
@@ -268,7 +268,13 @@ public:
{
mpGraphics->performFlush();
Stop();
SetPriority(TaskPriority::HIGHEST);
// tdf#157312 Don't change priority
// Instances of this class are constructed with
// TaskPriority::POST_PAINT, but then it was set to
// TaskPriority::HIGHEST when reused. Flushing
// seems to be expensive (at least with Skia/Metal) so keep the
// existing priority when reused.
SetPriority(TaskPriority::POST_PAINT);
}
};