commit 83cadf937ff9a0fd4ca24627673d45d1bab8c914
author Caolán McNamara <caolanm@redhat.com> Mon Oct 11 14:23:04 2021 +0100
committer Caolán McNamara <caolanm@redhat.com> Mon Oct 11 18:21:49 2021 +0200
tree 054a618d7bb37d2343e65a75d2a730dee2628a10
parent 8fb4889840a6c79f6bad7bb7a322d86b751cdd93

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/+/123413
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

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;
            }
        }