https://bugs.documentfoundation.org/show_bug.cgi?id=158397

--- Comment #11 from Patrick Luby (volunteer) <guibomac...@gmail.com> ---
(In reply to Patrick Luby (volunteer) from comment #10)
> Created attachment 195052 [details]
> Snapshot of CPU Profile in Instruments application when using Skia/Raster

In the CPU Profile I ran, the 2 selected lines were using 2/3rds of total CPU
usage. So I commented out FuDraw::ForcePointer() using the following debug
patch and that made Skia/Raster behave very closely to Skia/Metal i.e. only a
brief pause when moving the mouse and no pause when not moving the mouse.

I suspect that even without the FuDraw::ForcePointer() code, there are still
other places using excessive CPU, but FuDraw::ForcePointer() appears to be the
biggest contributor to this bug so I'll see if I can find a fix for that first:

diff --git a/sd/source/ui/func/fudraw.cxx b/sd/source/ui/func/fudraw.cxx
index 3e36d5d78f76..cf2bff5e6488 100644
--- a/sd/source/ui/func/fudraw.cxx
+++ b/sd/source/ui/func/fudraw.cxx
@@ -450,6 +450,8 @@ void FuDraw::Activate()
  */
 void FuDraw::ForcePointer(const MouseEvent* pMEvt)
 {
+(void)pMEvt;
+#if 0
     Point aPnt;
     sal_uInt16 nModifier = 0;
     bool bLeftDown = false;
@@ -574,6 +576,7 @@ void FuDraw::ForcePointer(const MouseEvent* pMEvt)
         mpWindow->SetPointer(mpView->GetPreferredPointer(
                             aPnt, mpWindow->GetOutDev(), nModifier,
bLeftDown));
     }
+#endif
 }

 /**

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to