vcl/source/window/window.cxx |    3 ---
 1 file changed, 3 deletions(-)

New commits:
commit 024053c0c140e1d167a0276f1885a21a160b45ac
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Mon May 26 10:40:35 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Thu Jun 19 21:32:21 2025 +0200

    vcl a11y: Drop obsolete comment in Window::dispose
    
    This is obsolete after
    
        Change-Id: I25de1ab0f8b47b12346abd01bad05cb54fb75f52
        Author: Michael Weghorn <m.wegh...@posteo.de>
        Date:   Mon May 26 09:45:11 2025 +0200
    
            vcl a11y: Let vcl::Window create accessible directly, drop a11y 
factory
    
    and
    
        Change-Id: Ided896abdc84cfbbf883621020169699a37cf6df
        Author: Michael Weghorn <m.wegh...@posteo.de>
        Date:   Mon May 26 10:29:46 2025 +0200
    
            a11y: Drop a11y logic from VCLXWindow
    
    because the VCLXWindow can no longer be the XAccessible of
    the vcl::Window.
    
    (Leave the order as is however, as there is no reason to change
    it either.)
    
    Change-Id: I1d8dc25476d616dcb2f0e2f5a7562a5e35e79ebb
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/185845
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>
    Tested-by: Jenkins

diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx
index 8e08e461fd35..a11bfddff75e 100644
--- a/vcl/source/window/window.cxx
+++ b/vcl/source/window/window.cxx
@@ -214,9 +214,6 @@ void Window::dispose()
     if ( pWrapper )
         pWrapper->WindowDestroyed( this );
 
-    // MT: Must be called after WindowDestroyed!
-    // Otherwise, if the accessible is a VCLXWindow, it will try to destroy 
this window again!
-    // But accessibility implementations from applications need this dispose.
     if ( mpWindowImpl->mxAccessible.is() )
     {
         Reference< XComponent> xC( mpWindowImpl->mxAccessible, UNO_QUERY );

Reply via email to