accessibility/Library_acc.mk                        |    5 -----
 accessibility/source/helper/acc_factory.cxx         |   11 -----------
 include/vcl/accessiblefactory.hxx                   |    5 -----
 solenv/clang-format/excludelist                     |   16 ++++++++--------
 svtools/Library_svt.mk                              |    4 ++++
 svtools/source/control/accessibletabbar.cxx         |    5 +++--
 svtools/source/control/accessibletabbar.hxx         |    4 ++--
 svtools/source/control/accessibletabbarbase.cxx     |    5 +++--
 svtools/source/control/accessibletabbarpage.cxx     |    3 ++-
 svtools/source/control/accessibletabbarpage.hxx     |    4 ++--
 svtools/source/control/accessibletabbarpagelist.cxx |    3 ++-
 svtools/source/control/accessibletabbarpagelist.hxx |    6 ++++--
 svtools/source/control/tabbar.cxx                   |    7 ++-----
 13 files changed, 32 insertions(+), 46 deletions(-)

New commits:
commit a7cc36dad9f37dcd74fe03ea08887a7732f08ff0
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Mon Dec 16 14:05:35 2024 +0100
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Mon Dec 16 19:08:06 2024 +0100

    a11y: Move TabBar a11y classses to svtools, no longer use factory
    
    Move AccessibleTabBar and related classes to
    implement accessibility for the TabBar control
    to svtools, where the TabBar code is also located.
    (This matches how it's organized for the Ruler control
    whose a11y class AccessibleRuler is also in svtools.)
    
    This removes the last dependency of the accessibility
    module on svtools, so drop that from accessibility/Library_acc.mk.
    
    Instead of using the AccessibleFactory to
    create an instance in TabBar::CreateAccessible,
    just call the AccessibleTabBar ctor directly.
    
    Change-Id: Id6a8852de930ffe7fe7509f84f33861d274120dd
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/178588
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>
    Tested-by: Jenkins

diff --git a/accessibility/Library_acc.mk b/accessibility/Library_acc.mk
index aafe63e86f9e..61bf4d590bb1 100644
--- a/accessibility/Library_acc.mk
+++ b/accessibility/Library_acc.mk
@@ -32,7 +32,6 @@ $(eval $(call gb_Library_use_libraries,acc,\
     i18nlangtag \
     sot \
     svl \
-    svt \
     tl \
     utl \
     vcl \
@@ -65,10 +64,6 @@ $(eval $(call gb_Library_add_exception_objects,acc,\
     accessibility/source/extended/AccessibleIconView \
     accessibility/source/extended/accessiblelistbox \
     accessibility/source/extended/accessiblelistboxentry \
-    accessibility/source/extended/accessibletabbar \
-    accessibility/source/extended/accessibletabbarbase \
-    accessibility/source/extended/accessibletabbarpage \
-    accessibility/source/extended/accessibletabbarpagelist \
     accessibility/source/extended/accessibletablistbox \
     accessibility/source/extended/accessibletablistboxtable \
     accessibility/source/extended/textwindowaccessibility \
diff --git a/accessibility/source/helper/acc_factory.cxx 
b/accessibility/source/helper/acc_factory.cxx
index 75d4bd13e6db..7586e3322082 100644
--- a/accessibility/source/helper/acc_factory.cxx
+++ b/accessibility/source/helper/acc_factory.cxx
@@ -44,7 +44,6 @@
 #include <extended/AccessibleBrowseBox.hxx>
 #include <extended/accessibleiconchoicectrl.hxx>
 #include <extended/AccessibleIconView.hxx>
-#include <extended/accessibletabbar.hxx>
 #include <extended/accessiblelistbox.hxx>
 #include <extended/AccessibleBrowseBoxHeaderBar.hxx>
 #include <extended/textwindowaccessibility.hxx>
@@ -138,11 +137,6 @@ public:
             const css::uno::Reference< css::accessibility::XAccessible >& 
_xParent
         ) const override;
 
-    virtual css::uno::Reference< css::accessibility::XAccessible >
-        createAccessibleTabBar(
-            TabBar& _rTabBar
-        ) const override;
-
     virtual css::uno::Reference< css::accessibility::XAccessibleContext >
         createAccessibleTextWindowContext(
             vcl::Window* pWindow, TextEngine& rEngine, TextView& rView
@@ -356,11 +350,6 @@ Reference< XAccessible > 
AccessibleFactory::createAccessibleIconChoiceCtrl(
     return new AccessibleIconChoiceCtrl( _rIconCtrl, _xParent );
 }
 
-Reference< XAccessible > AccessibleFactory::createAccessibleTabBar( TabBar& 
_rTabBar ) const
-{
-    return new AccessibleTabBar( &_rTabBar );
-}
-
 Reference< XAccessibleContext > 
AccessibleFactory::createAccessibleTextWindowContext(
     vcl::Window* pWindow, TextEngine& rEngine, TextView& rView) const
 {
diff --git a/include/vcl/accessiblefactory.hxx 
b/include/vcl/accessiblefactory.hxx
index 84b508910bb9..0620964230cd 100644
--- a/include/vcl/accessiblefactory.hxx
+++ b/include/vcl/accessiblefactory.hxx
@@ -106,11 +106,6 @@ namespace vcl
                 const css::uno::Reference< css::accessibility::XAccessible >& 
_xParent
             ) const = 0;
 
-        virtual css::uno::Reference< css::accessibility::XAccessible >
-            createAccessibleTabBar(
-                TabBar& _rTabBar
-            ) const = 0;
-
         virtual css::uno::Reference< css::accessibility::XAccessibleContext >
             createAccessibleTextWindowContext(
                 vcl::Window* pWindow, TextEngine& rEngine, TextView& rView
diff --git a/solenv/clang-format/excludelist b/solenv/clang-format/excludelist
index cda45a1ab572..233ee1be5db5 100644
--- a/solenv/clang-format/excludelist
+++ b/solenv/clang-format/excludelist
@@ -37,10 +37,6 @@ accessibility/inc/extended/accessibleiconchoicectrl.hxx
 accessibility/inc/extended/accessibleiconchoicectrlentry.hxx
 accessibility/inc/extended/accessiblelistbox.hxx
 accessibility/inc/extended/accessiblelistboxentry.hxx
-accessibility/inc/extended/accessibletabbar.hxx
-accessibility/inc/extended/accessibletabbarbase.hxx
-accessibility/inc/extended/accessibletabbarpage.hxx
-accessibility/inc/extended/accessibletabbarpagelist.hxx
 accessibility/inc/extended/accessibletablistbox.hxx
 accessibility/inc/extended/accessibletablistboxtable.hxx
 accessibility/inc/extended/textwindowaccessibility.hxx
@@ -85,10 +81,6 @@ accessibility/source/extended/accessibleiconchoicectrl.cxx
 accessibility/source/extended/accessibleiconchoicectrlentry.cxx
 accessibility/source/extended/accessiblelistbox.cxx
 accessibility/source/extended/accessiblelistboxentry.cxx
-accessibility/source/extended/accessibletabbar.cxx
-accessibility/source/extended/accessibletabbarbase.cxx
-accessibility/source/extended/accessibletabbarpage.cxx
-accessibility/source/extended/accessibletabbarpagelist.cxx
 accessibility/source/extended/accessibletablistbox.cxx
 accessibility/source/extended/accessibletablistboxtable.cxx
 accessibility/source/extended/textwindowaccessibility.cxx
@@ -11052,6 +11044,14 @@ svtools/source/config/printoptions.cxx
 svtools/source/config/test/test.cxx
 svtools/source/control/accessibleruler.cxx
 svtools/source/control/accessibleruler.hxx
+svtools/source/control/accessibletabbar.cxx
+svtools/source/control/accessibletabbar.hxx
+svtools/source/control/accessibletabbarbase.cxx
+svtools/source/control/accessibletabbarbase.hxx
+svtools/source/control/accessibletabbarpage.cxx
+svtools/source/control/accessibletabbarpage.hxx
+svtools/source/control/accessibletabbarpagelist.cxx
+svtools/source/control/accessibletabbarpagelist.hxx
 svtools/source/control/asynclink.cxx
 svtools/source/control/ctrlbox.cxx
 svtools/source/control/ctrltool.cxx
diff --git a/svtools/Library_svt.mk b/svtools/Library_svt.mk
index 14658bbe9ea3..5782d70261ac 100644
--- a/svtools/Library_svt.mk
+++ b/svtools/Library_svt.mk
@@ -89,6 +89,10 @@ $(eval $(call gb_Library_add_exception_objects,svt,\
     svtools/source/config/printoptions \
     svtools/source/contnr/querydelete \
     svtools/source/control/accessibleruler \
+    svtools/source/control/accessibletabbar \
+    svtools/source/control/accessibletabbarbase \
+    svtools/source/control/accessibletabbarpage \
+    svtools/source/control/accessibletabbarpagelist \
     svtools/source/control/asynclink \
     svtools/source/control/collatorres \
     svtools/source/control/ctrlbox \
diff --git a/accessibility/source/extended/accessibletabbar.cxx 
b/svtools/source/control/accessibletabbar.cxx
similarity index 99%
rename from accessibility/source/extended/accessibletabbar.cxx
rename to svtools/source/control/accessibletabbar.cxx
index ede35977d93a..92bab62c0143 100644
--- a/accessibility/source/extended/accessibletabbar.cxx
+++ b/svtools/source/control/accessibletabbar.cxx
@@ -17,9 +17,10 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
-#include <extended/accessibletabbar.hxx>
+#include "accessibletabbar.hxx"
+#include "accessibletabbarpagelist.hxx"
+
 #include <svtools/tabbar.hxx>
-#include <extended/accessibletabbarpagelist.hxx>
 #include <com/sun/star/accessibility/AccessibleEventId.hpp>
 #include <com/sun/star/accessibility/AccessibleRole.hpp>
 #include <com/sun/star/accessibility/AccessibleStateType.hpp>
diff --git a/accessibility/inc/extended/accessibletabbar.hxx 
b/svtools/source/control/accessibletabbar.hxx
similarity index 98%
rename from accessibility/inc/extended/accessibletabbar.hxx
rename to svtools/source/control/accessibletabbar.hxx
index 6d75c83a7e23..5b6f951fa14a 100644
--- a/accessibility/inc/extended/accessibletabbar.hxx
+++ b/svtools/source/control/accessibletabbar.hxx
@@ -19,14 +19,14 @@
 
 #pragma once
 
+#include "accessibletabbarbase.hxx"
+
 #include <com/sun/star/accessibility/XAccessible.hpp>
 #include <com/sun/star/lang/XServiceInfo.hpp>
 #include <cppuhelper/implbase.hxx>
-#include <extended/accessibletabbarbase.hxx>
 
 #include <vector>
 
-
 namespace accessibility
 {
 
diff --git a/accessibility/source/extended/accessibletabbarbase.cxx 
b/svtools/source/control/accessibletabbarbase.cxx
similarity index 96%
rename from accessibility/source/extended/accessibletabbarbase.cxx
rename to svtools/source/control/accessibletabbarbase.cxx
index 726699757e5c..08142db60f6f 100644
--- a/accessibility/source/extended/accessibletabbarbase.cxx
+++ b/svtools/source/control/accessibletabbarbase.cxx
@@ -17,8 +17,9 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
-#include <extended/accessibletabbarbase.hxx>
-#include <extended/accessibletabbarpagelist.hxx>
+#include "accessibletabbarbase.hxx"
+#include "accessibletabbarpagelist.hxx"
+
 #include <svtools/tabbar.hxx>
 #include <vcl/vclevent.hxx>
 
diff --git a/accessibility/inc/extended/accessibletabbarbase.hxx 
b/svtools/source/control/accessibletabbarbase.hxx
similarity index 100%
rename from accessibility/inc/extended/accessibletabbarbase.hxx
rename to svtools/source/control/accessibletabbarbase.hxx
diff --git a/accessibility/source/extended/accessibletabbarpage.cxx 
b/svtools/source/control/accessibletabbarpage.cxx
similarity index 99%
rename from accessibility/source/extended/accessibletabbarpage.cxx
rename to svtools/source/control/accessibletabbarpage.cxx
index f2de152f871f..5a15d33b6309 100644
--- a/accessibility/source/extended/accessibletabbarpage.cxx
+++ b/svtools/source/control/accessibletabbarpage.cxx
@@ -17,7 +17,8 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
-#include <extended/accessibletabbarpage.hxx>
+#include "accessibletabbarpage.hxx"
+
 #include <svtools/tabbar.hxx>
 #include <com/sun/star/accessibility/AccessibleEventId.hpp>
 #include <com/sun/star/accessibility/AccessibleRole.hpp>
diff --git a/accessibility/inc/extended/accessibletabbarpage.hxx 
b/svtools/source/control/accessibletabbarpage.hxx
similarity index 98%
rename from accessibility/inc/extended/accessibletabbarpage.hxx
rename to svtools/source/control/accessibletabbarpage.hxx
index a8266b55c2a5..6a7198dff737 100644
--- a/accessibility/inc/extended/accessibletabbarpage.hxx
+++ b/svtools/source/control/accessibletabbarpage.hxx
@@ -19,11 +19,11 @@
 
 #pragma once
 
+#include "accessibletabbarbase.hxx"
+
 #include <com/sun/star/accessibility/XAccessible.hpp>
 #include <com/sun/star/lang/XServiceInfo.hpp>
 #include <cppuhelper/implbase.hxx>
-#include <extended/accessibletabbarbase.hxx>
-
 
 namespace accessibility
 {
diff --git a/accessibility/source/extended/accessibletabbarpagelist.cxx 
b/svtools/source/control/accessibletabbarpagelist.cxx
similarity index 99%
rename from accessibility/source/extended/accessibletabbarpagelist.cxx
rename to svtools/source/control/accessibletabbarpagelist.cxx
index 4bde87630738..9e1edc0efa1b 100644
--- a/accessibility/source/extended/accessibletabbarpagelist.cxx
+++ b/svtools/source/control/accessibletabbarpagelist.cxx
@@ -17,7 +17,8 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
-#include <extended/accessibletabbarpagelist.hxx>
+#include "accessibletabbarpagelist.hxx"
+
 #include <svtools/tabbar.hxx>
 #include <com/sun/star/accessibility/AccessibleEventId.hpp>
 #include <com/sun/star/accessibility/AccessibleRole.hpp>
diff --git a/accessibility/inc/extended/accessibletabbarpagelist.hxx 
b/svtools/source/control/accessibletabbarpagelist.hxx
similarity index 98%
rename from accessibility/inc/extended/accessibletabbarpagelist.hxx
rename to svtools/source/control/accessibletabbarpagelist.hxx
index 950a37167086..88bb87b0a96a 100644
--- a/accessibility/inc/extended/accessibletabbarpagelist.hxx
+++ b/svtools/source/control/accessibletabbarpagelist.hxx
@@ -19,12 +19,14 @@
 
 #pragma once
 
+
+#include "accessibletabbarbase.hxx"
+#include "accessibletabbarpage.hxx"
+
 #include <com/sun/star/accessibility/XAccessible.hpp>
 #include <com/sun/star/accessibility/XAccessibleSelection.hpp>
 #include <com/sun/star/lang/XServiceInfo.hpp>
 #include <cppuhelper/implbase.hxx>
-#include <extended/accessibletabbarbase.hxx>
-#include <extended/accessibletabbarpage.hxx>
 
 #include <vector>
 
diff --git a/svtools/source/control/tabbar.cxx 
b/svtools/source/control/tabbar.cxx
index 81780ad8c92f..6f38a6072e67 100644
--- a/svtools/source/control/tabbar.cxx
+++ b/svtools/source/control/tabbar.cxx
@@ -17,6 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
+#include "accessibletabbar.hxx"
 
 #include <svtools/tabbar.hxx>
 #include <tools/time.hxx>
@@ -30,8 +31,6 @@
 #include <vcl/event.hxx>
 #include <vcl/settings.hxx>
 #include <vcl/commandevent.hxx>
-#include <vcl/svtaccessiblefactory.hxx>
-#include <vcl/accessiblefactory.hxx>
 #include <vcl/ptrstyle.hxx>
 #include <vcl/weldutils.hxx>
 #include <svtools/svtresid.hxx>
@@ -511,8 +510,6 @@ struct TabBar_Impl
     ScopedVclPtr<TabBarEdit>    mxEdit;
     std::vector<ImplTabBarItem> maItemList;
 
-    vcl::AccessibleFactoryAccess  maAccessibleFactory;
-
     sal_uInt16 getItemSize() const
     {
         return static_cast<sal_uInt16>(maItemList.size());
@@ -2536,7 +2533,7 @@ void TabBar::SetAddButtonEnabled(bool bAddButtonEnabled)
 
 css::uno::Reference<css::accessibility::XAccessible> TabBar::CreateAccessible()
 {
-    return 
mpImpl->maAccessibleFactory.getFactory().createAccessibleTabBar(*this);
+    return new accessibility::AccessibleTabBar(this);
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */

Reply via email to