commit 61ef0a842b42ac998a533e8e1ba1e00e02cd973d
author Caolán McNamara <caolanm@redhat.com> Mon Oct 11 14:23:04 2021 +0100
committer Julien Nabet <serval2412@yahoo.fr> Mon Oct 11 20:57:19 2021 +0200
tree b4b17dd100f18627dc5028144d414f31abdff00e
parent 74aef3f0af59008aa9390bad0b24f8fce0632c87

tdf#145033 remove old focus handler so no double-connect attempt is made

doesn't hurt in this case, but we assert because it has been a fairly
common problem in the past

Change-Id: Id312c8d2693df803962c265a9a49926d8df04cc0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123395
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

extensions/source/propctrlr/browserline.cxx

diff --git a/extensions/source/propctrlr/browserline.cxx b/extensions/source/propctrlr/browserline.cxx
index 8618ecf..d4770ba 100644
--- a/extensions/source/propctrlr/browserline.cxx
+++ b/extensions/source/propctrlr/browserline.cxx
@@ -346,6 +346,7 @@ namespace pcr
            if (m_pBrowseButton)
            {
                m_pBrowseButton->hide();
                m_pBrowseButton->connect_focus_in(Link<weld::Widget&, void>());
                m_pBrowseButton = nullptr;
            }
        }
@@ -354,6 +355,7 @@ namespace pcr
            if (m_pAdditionalBrowseButton)
            {
                m_pAdditionalBrowseButton->hide();
                m_pAdditionalBrowseButton->connect_focus_in(Link<weld::Widget&, void>());
                m_pAdditionalBrowseButton = nullptr;
            }
        }