vcl/skia/gdiimpl.cxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit ad59ecf37f847295e99069281d94518f5afedf90
Author: Patrick Luby <guibmac...@gmail.com>
AuthorDate: Tue Jun 17 14:19:06 2025 -0400
Commit: Patrick Luby <guibomac...@gmail.com>
CommitDate: Wed Jun 18 19:51:11 2025 +0200
tdf#167059 Revert remaining use of POST_PAINT priority for Skia paint timer
In commit f4c2c7c79cfe4464ac596afda37b8904d06969db and in
commit 5a38e4f9798c5ff247aa57581adf2671485627fd, the Skia paint
timer's priority was changed from HIGHEST to POST_PAINT in order
fix blocking on the main thread when using Skia/Metal on macOS.
However, using lower priorities ended up causing many of the
bugs that were fixed on macOS to occur on Windows and Linux so
the macOS fixes were reverted and added back in #ifdef MACOSX
blocks but two of the changes from HIGHEST to POST_PAINT were
accidently not reverted so revert them on Windows and Linux.
Change-Id: Ic9f184d324ad0760f2c51f9f12aee4dba195ea92
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/186631
Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
Tested-by: Jenkins
Reviewed-by: Patrick Luby <guibomac...@gmail.com>
diff --git a/vcl/skia/gdiimpl.cxx b/vcl/skia/gdiimpl.cxx
index ee25f385ceca..10368a347d64 100644
--- a/vcl/skia/gdiimpl.cxx
+++ b/vcl/skia/gdiimpl.cxx
@@ -251,7 +251,7 @@ public:
SetPriority(TaskPriority::SKIA_FLUSH);
#else
// We don't want to be swapping before we've painted.
- SetPriority(TaskPriority::POST_PAINT);
+ SetPriority(TaskPriority::HIGHEST);
#endif
}
#ifndef NDEBUG
@@ -328,7 +328,7 @@ void SkiaSalGraphicsImpl::createSurface()
// tasks on macOS
mFlush->SetPriority(TaskPriority::SKIA_FLUSH);
#else
- mFlush->SetPriority(TaskPriority::POST_PAINT);
+ mFlush->SetPriority(TaskPriority::HIGHEST);
#endif
}
