vcl/skia/gdiimpl.cxx |    8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

New commits:
commit 45c3164e569ff06160d64f085a19618ed70c13ee
Author:     Patrick Luby <guibmac...@gmail.com>
AuthorDate: Sat Nov 9 18:27:35 2024 -0500
Commit:     Michael Stahl <michael.st...@allotropia.de>
CommitDate: Mon Nov 11 11:23:53 2024 +0100

    tdf#163734 Revert commit f4c2c7c79cfe4464ac596afda37b8904d06969db
    
    Commit f4c2c7c79cfe4464ac596afda37b8904d06969db lowered the Skia
    flushing priority to solve tdf#157312. But the commit caused the
    "window will dock in rectangle" border to not be drawn when dragging
    a dockable window over a dockable position.
    
    Surprisingly, tdf#157312 does not reoccur without the commit so
    it appears that the commit is now unnecessary.
    
    Change-Id: Ibd74ef94e8448cf32fd690d7a26dd098191040b8
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176341
    Tested-by: Jenkins
    Reviewed-by: Patrick Luby <guibomac...@gmail.com>
    (cherry picked from commit 5a38e4f9798c5ff247aa57581adf2671485627fd)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/176346
    Reviewed-by: Michael Stahl <michael.st...@allotropia.de>

diff --git a/vcl/skia/gdiimpl.cxx b/vcl/skia/gdiimpl.cxx
index f7a9dda2db84..94ad83038906 100644
--- a/vcl/skia/gdiimpl.cxx
+++ b/vcl/skia/gdiimpl.cxx
@@ -269,13 +269,7 @@ public:
     {
         mpGraphics->performFlush();
         Stop();
-        // 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);
+        SetPriority(TaskPriority::HIGHEST);
     }
 };
 

Reply via email to