[Libreoffice-commits] core.git: Branch 'libreoffice-4-4' - sw/source

Michael Stahl mstahl at redhat.com
Fri Dec 19 05:11:24 PST 2014


 sw/source/core/access/accmap.cxx |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 00428a15f0e7346af5da80563197950cf6439297
Author: Michael Stahl <mstahl at redhat.com>
Date:   Thu Dec 18 21:52:51 2014 +0100

    fdo#87199: sw: fix root cause of a11y crash when merging cells
    
    Commit f9eff2a402a4cd28d7dbfb6ce27cbf96b31e576f is not quite right
    because it will leave the mpNext chain unreachable; that could perhaps
    be imporoved by calling RemoveFromLayout(), but...
    
    Actually the problem is basically that one of the deleted SwCellFrms
    points to a SwTableBox with getRowSpan() -1 (because it has been merged)
    and thus IsInCoveredCell() returns true and that causes ~SwCellFrm() to
    skip disposing the SwAccessible stuff, so the SwCellFrm is still
    contained in the SwAccessibleMap.
    
    Because it's rather hard to prevent this sort of thing in general,
    better change SwAccessibleMap::Dispose() to assume that if it found its
    way into the SwAccessibleMap it should be disposed and removed.
    
    Change-Id: Ib4cec6924cb026ae30bdac6857957adf237b4d70
    (cherry picked from commit 296e8b597c141b6b54cbf943871d6a6820c1779d)
    Reviewed-on: https://gerrit.libreoffice.org/13532
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/core/access/accmap.cxx b/sw/source/core/access/accmap.cxx
index 1d94b37..35250df 100644
--- a/sw/source/core/access/accmap.cxx
+++ b/sw/source/core/access/accmap.cxx
@@ -2270,7 +2270,9 @@ void SwAccessibleMap::Dispose( const SwFrm *pFrm,
     OSL_ENSURE( !aFrmOrObj.GetSwFrm() || aFrmOrObj.GetSwFrm()->IsAccessibleFrm(),
             "non accessible frame should be disposed" );
 
-    if( aFrmOrObj.IsAccessible( GetShell()->IsPreview() ) )
+    if (aFrmOrObj.IsAccessible( GetShell()->IsPreview() )
+            // fdo#87199 dispose the darn thing if it ever was accessible
+        || (pFrm && mpFrmMap && mpFrmMap->find(pFrm) != mpFrmMap->end()))
     {
         ::rtl::Reference< SwAccessibleContext > xAccImpl;
         ::rtl::Reference< SwAccessibleContext > xParentAccImpl;


More information about the Libreoffice-commits mailing list