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

Caolán McNamara (via logerrit) logerrit at kemper.freedesktop.org
Tue May 14 16:33:08 UTC 2019


 vcl/source/window/dialog.cxx |    8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

New commits:
commit 4cac171dd6c82f55624ba6e7185d69bcb5507cb1
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Tue May 14 16:01:58 2019 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Tue May 14 18:32:05 2019 +0200

    Resolves: tdf#125266 hang on selecting read online help
    
    Change-Id: I46d14df66c529495a7e9f1596fcdbb0b093810c5
    Reviewed-on: https://gerrit.libreoffice.org/72298
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/vcl/source/window/dialog.cxx b/vcl/source/window/dialog.cxx
index bd17f377043d..21957995644e 100644
--- a/vcl/source/window/dialog.cxx
+++ b/vcl/source/window/dialog.cxx
@@ -1576,8 +1576,12 @@ void TopLevelWindowLocker::incBusy(const weld::Window* pIgnore)
     vcl::Window *pTopWin = Application::GetFirstTopLevelWindow();
     while (pTopWin)
     {
-        if (pTopWin->GetFrameWeld() != pIgnore)
-            aTopLevels.push_back(pTopWin);
+        vcl::Window* pCandidate = pTopWin;
+        if (pCandidate->GetType() == WindowType::BORDERWINDOW)
+            pCandidate = pCandidate->GetWindow(GetWindowType::FirstChild);
+        // tdf#125266 ignore HelpTextWindows
+        if (pCandidate && pCandidate->GetType() != WindowType::HELPTEXTWINDOW && pCandidate->GetFrameWeld() != pIgnore)
+            aTopLevels.push_back(pCandidate);
         pTopWin = Application::GetNextTopLevelWindow(pTopWin);
     }
     for (auto& a : aTopLevels)


More information about the Libreoffice-commits mailing list