focus rect in treeview doesn't match the selected area

regression from...

commit 753084a3f73bb8a88412feac7aa40a2750e6e9f3
Date:   Wed Dec 11 12:56:57 2019 +0200

loplugin:singlevalfields

which detected that m_bIsCellFocusEnabled is always false but inadvertently
make a change of

-void SvImpLBox::CalcCellFocusRect( SvTreeListEntry const * pEntry, tools::Rectangle& rRect )
+void SvImpLBox::CalcCellFocusRect( tools::Rectangle& rRect )
{
-    if ( !(pEntry && m_bIsCellFocusEnabled) )
-        return;

which treats m_bIsCellFocusEnabled as always true. Always false would make
CalcCellFocusRect a no-op so just remove the method to fix things.

Change-Id: I46bfa122058a9e4384d2b1e2dc9e844ada22cc69
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95652
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

1 files changed, 0 insertions, 2 deletions

diff --git a/vcl/inc/svimpbox.hxx b/vcl/inc/svimpbox.hxx
index 5d23b3f2c46b..c5a51143f141 100644
--- a/vcl/inc/svimpbox.hxx
+++ b/vcl/inc/svimpbox.hxx
@@ -189,8 +189,6 @@ private:
     void                UpdateContextBmpWidthMax( SvTreeListEntry const * pEntry );
     void                UpdateContextBmpWidthVectorFromMovedEntry( SvTreeListEntry* pEntry );
 
-    void                CalcCellFocusRect( tools::Rectangle& rRect );
-
     void ExpandAll();
     void CollapseTo(SvTreeListEntry* pParentToCollapse);