[Libreoffice-commits] core.git: vcl/inc vcl/source

[Noel Grandin (via logerrit)]

 vcl/inc/dndeventdispatcher.hxx           |    2 +-
 vcl/source/window/dndeventdispatcher.cxx |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 3394fe4f19def29e314798fa64235b28d9df50a6
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Tue Jul 18 15:04:12 2023 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Tue Jul 18 21:08:00 2023 +0200

    fix DNDEventDispatcher::findTopLevelWindow
    
    for the unlikely case of antiparallel desktop (whatever that is)
    
    regression from
        commit ea05f2aa4e28fd40238c4962e9dcbbf0d873cabd
        Author: Norbert Thiebaud <nthieb...@gmail.com>
        Date:   Thu Oct 14 22:12:16 2010 -0500
        migrate OClearableGuard(SoalrMutex) to SolarMutexGuard
    
    when the code findTopLevelWindow was extracted into its own method,
    the location parameter was not made a reference, which is necessary to
    propogate the update that occurs in OutputDevice::ReMirror
    
    Change-Id: I37c18497660cd157ca4e4cfb1cbb60ed07e11da5
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154586
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/vcl/inc/dndeventdispatcher.hxx b/vcl/inc/dndeventdispatcher.hxx
index 9756056da3bd..0b3f58560282 100644
--- a/vcl/inc/dndeventdispatcher.hxx
+++ b/vcl/inc/dndeventdispatcher.hxx
@@ -42,7 +42,7 @@ class DNDEventDispatcher final : public 
::cppu::WeakImplHelper<
     std::recursive_mutex m_aMutex;
     css::uno::Sequence< css::datatransfer::DataFlavor > m_aDataFlavorList;
 
-    vcl::Window* findTopLevelWindow(Point location);
+    vcl::Window* findTopLevelWindow(Point& location);
     /*
      * fire the events on the dnd listener container of the specified window
      */
diff --git a/vcl/source/window/dndeventdispatcher.cxx 
b/vcl/source/window/dndeventdispatcher.cxx
index a5a89928e616..c57841c1fdfa 100644
--- a/vcl/source/window/dndeventdispatcher.cxx
+++ b/vcl/source/window/dndeventdispatcher.cxx
@@ -42,7 +42,7 @@ DNDEventDispatcher::~DNDEventDispatcher()
     designate_currentwindow(nullptr);
 }
 
-vcl::Window* DNDEventDispatcher::findTopLevelWindow(Point location)
+vcl::Window* DNDEventDispatcher::findTopLevelWindow(Point& location)
 {
     SolarMutexGuard aSolarGuard;