include/sfx2/devtools/DevelopmentToolDockingWindow.hxx | 9 include/sfx2/devtools/ObjectInspectorTreeHandler.hxx | 18 - include/sfx2/devtools/ObjectInspectorWidgets.hxx | 49 +++++ sfx2/source/devtools/DevelopmentToolDockingWindow.cxx | 19 -- sfx2/source/devtools/ObjectInspectorTreeHandler.cxx | 155 ++++++++--------- 5 files changed, 132 insertions(+), 118 deletions(-)
New commits: commit 173697822d6a598461f79da2e2f77f24723d40ab Author: Tomaž Vajngerl <tomaz.vajng...@collabora.co.uk> AuthorDate: Mon Mar 15 14:56:57 2021 +0900 Commit: Tomaž Vajngerl <qui...@gmail.com> CommitDate: Tue Mar 16 03:41:15 2021 +0100 devtools: put ObjectInspector widgets into it's own class This is needed so they can be shared between ObjectInspectorHandler and DevelopmentToolDockingWindow, otherwise we have to add each widget as a parameter to the constructor and for each make a referenced member. This worked for a while but with more a nd ore widgets it is becoming ugly. Change-Id: I8f17e4593b04efaf85ef50d0639c306c204ce92a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112547 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <qui...@gmail.com> diff --git a/include/sfx2/devtools/DevelopmentToolDockingWindow.hxx b/include/sfx2/devtools/DevelopmentToolDockingWindow.hxx index 1d05948cd51f..656e46fedada 100644 --- a/include/sfx2/devtools/DevelopmentToolDockingWindow.hxx +++ b/include/sfx2/devtools/DevelopmentToolDockingWindow.hxx @@ -15,6 +15,7 @@ #include <vcl/customweld.hxx> #include <vcl/weld.hxx> +#include <sfx2/devtools/ObjectInspectorWidgets.hxx> #include <sfx2/devtools/DocumentModelTreeHandler.hxx> #include <sfx2/devtools/ObjectInspectorTreeHandler.hxx> @@ -34,15 +35,9 @@ class SFX2_DLLPUBLIC DevelopmentToolDockingWindow final : public SfxDockingWindow { private: - std::unique_ptr<weld::Label> mpClassNameLabel; - std::unique_ptr<weld::TreeView> mpInterfacesTreeView; - std::unique_ptr<weld::TreeView> mpServicesTreeView; - std::unique_ptr<weld::TreeView> mpPropertiesTreeView; - std::unique_ptr<weld::TreeView> mpMethodsTreeView; + std::unique_ptr<ObjectInspectorWidgets> mpObjectInspectorWidgets; std::unique_ptr<weld::TreeView> mpDocumentModelTreeView; std::unique_ptr<weld::ToggleButton> mpSelectionToggle; - std::unique_ptr<weld::Toolbar> mpObjectInspectorToolbar; - std::unique_ptr<weld::Notebook> mpObjectInspectorNotebook; // Reference to the root object for the current document css::uno::Reference<css::uno::XInterface> mxRoot; diff --git a/include/sfx2/devtools/ObjectInspectorTreeHandler.hxx b/include/sfx2/devtools/ObjectInspectorTreeHandler.hxx index 1d7000371eba..2cfb572eb3a2 100644 --- a/include/sfx2/devtools/ObjectInspectorTreeHandler.hxx +++ b/include/sfx2/devtools/ObjectInspectorTreeHandler.hxx @@ -19,6 +19,8 @@ #include <com/sun/star/uno/XInterface.hpp> #include <com/sun/star/uno/XComponentContext.hpp> +#include <sfx2/devtools/ObjectInspectorWidgets.hxx> + #include <memory> #include <deque> @@ -31,13 +33,7 @@ class ObjectInspectorTreeHandler { private: - std::unique_ptr<weld::TreeView>& mpInterfacesTreeView; - std::unique_ptr<weld::TreeView>& mpServicesTreeView; - std::unique_ptr<weld::TreeView>& mpPropertiesTreeView; - std::unique_ptr<weld::TreeView>& mpMethodsTreeView; - std::unique_ptr<weld::Label>& mpClassNameLabel; - std::unique_ptr<weld::Toolbar>& mpObjectInspectorToolbar; - std::unique_ptr<weld::Notebook>& mpObjectInspectorNotebook; + std::unique_ptr<ObjectInspectorWidgets>& mpObjectInspectorWidgets; // object stack to remember previously inspected objects so it is // possible to return back to them @@ -67,13 +63,7 @@ private: void updateBackButtonState(); public: - ObjectInspectorTreeHandler(std::unique_ptr<weld::TreeView>& pInterfacesTreeView, - std::unique_ptr<weld::TreeView>& pServicesTreeView, - std::unique_ptr<weld::TreeView>& pPropertiesTreeView, - std::unique_ptr<weld::TreeView>& pMethodsTreeView, - std::unique_ptr<weld::Label>& pClassNameLabel, - std::unique_ptr<weld::Toolbar>& pObjectInspectorToolbar, - std::unique_ptr<weld::Notebook>& pObjectInspectorNotebook); + ObjectInspectorTreeHandler(std::unique_ptr<ObjectInspectorWidgets>& pObjectInspectorWidgets); // callbacks when a node in the tree view is expanded DECL_LINK(ExpandingHandlerInterfaces, const weld::TreeIter&, bool); diff --git a/include/sfx2/devtools/ObjectInspectorWidgets.hxx b/include/sfx2/devtools/ObjectInspectorWidgets.hxx new file mode 100644 index 000000000000..6d7bf8cf9e9f --- /dev/null +++ b/include/sfx2/devtools/ObjectInspectorWidgets.hxx @@ -0,0 +1,49 @@ +/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */ +/* + * This file is part of the LibreOffice project. + * + * This Source Code Form is subject to the terms of the Mozilla Public + * License, v. 2.0. If a copy of the MPL was not distributed with this + * file, You can obtain one at http://mozilla.org/MPL/2.0/. + * + */ + +#pragma once + +#include <vcl/weld.hxx> + +struct ObjectInspectorWidgets +{ + ObjectInspectorWidgets(std::unique_ptr<weld::Builder>& rxBuilder) + : mpClassNameLabel(rxBuilder->weld_label("class_name_value_id")) + , mpInterfacesTreeView(rxBuilder->weld_tree_view("interfaces_treeview_id")) + , mpServicesTreeView(rxBuilder->weld_tree_view("services_treeview_id")) + , mpPropertiesTreeView(rxBuilder->weld_tree_view("properties_treeview_id")) + , mpMethodsTreeView(rxBuilder->weld_tree_view("methods_treeview_id")) + , mpToolbar(rxBuilder->weld_toolbar("object_inspector_toolbar")) + , mpNotebook(rxBuilder->weld_notebook("object_inspector_notebookbar")) + { + } + + ~ObjectInspectorWidgets() + { + // dispose welded objects + mpClassNameLabel.reset(); + mpInterfacesTreeView.reset(); + mpServicesTreeView.reset(); + mpPropertiesTreeView.reset(); + mpMethodsTreeView.reset(); + mpToolbar.reset(); + mpNotebook.reset(); + } + + std::unique_ptr<weld::Label> mpClassNameLabel; + std::unique_ptr<weld::TreeView> mpInterfacesTreeView; + std::unique_ptr<weld::TreeView> mpServicesTreeView; + std::unique_ptr<weld::TreeView> mpPropertiesTreeView; + std::unique_ptr<weld::TreeView> mpMethodsTreeView; + std::unique_ptr<weld::Toolbar> mpToolbar; + std::unique_ptr<weld::Notebook> mpNotebook; +}; + +/* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/devtools/DevelopmentToolDockingWindow.cxx b/sfx2/source/devtools/DevelopmentToolDockingWindow.cxx index 120c5b0ed23b..6fcbcc68812d 100644 --- a/sfx2/source/devtools/DevelopmentToolDockingWindow.cxx +++ b/sfx2/source/devtools/DevelopmentToolDockingWindow.cxx @@ -27,21 +27,13 @@ DevelopmentToolDockingWindow::DevelopmentToolDockingWindow(SfxBindings* pInputBi vcl::Window* pParent) : SfxDockingWindow(pInputBindings, pChildWindow, pParent, "DevelopmentTool", "sfx/ui/developmenttool.ui") - , mpClassNameLabel(m_xBuilder->weld_label("class_name_value_id")) - , mpInterfacesTreeView(m_xBuilder->weld_tree_view("interfaces_treeview_id")) - , mpServicesTreeView(m_xBuilder->weld_tree_view("services_treeview_id")) - , mpPropertiesTreeView(m_xBuilder->weld_tree_view("properties_treeview_id")) - , mpMethodsTreeView(m_xBuilder->weld_tree_view("methods_treeview_id")) + , mpObjectInspectorWidgets(new ObjectInspectorWidgets(m_xBuilder)) , mpDocumentModelTreeView(m_xBuilder->weld_tree_view("leftside_treeview_id")) , mpSelectionToggle(m_xBuilder->weld_toggle_button("selection_toggle")) - , mpObjectInspectorToolbar(m_xBuilder->weld_toolbar("object_inspector_toolbar")) - , mpObjectInspectorNotebook(m_xBuilder->weld_notebook("object_inspector_notebookbar")) , maDocumentModelTreeHandler( mpDocumentModelTreeView, pInputBindings->GetDispatcher()->GetFrame()->GetObjectShell()->GetBaseModel()) - , maObjectInspectorTreeHandler(mpInterfacesTreeView, mpServicesTreeView, mpPropertiesTreeView, - mpMethodsTreeView, mpClassNameLabel, mpObjectInspectorToolbar, - mpObjectInspectorNotebook) + , maObjectInspectorTreeHandler(mpObjectInspectorWidgets) { mpDocumentModelTreeView->connect_changed( LINK(this, DevelopmentToolDockingWindow, DocumentModelTreeViewSelectionHandler)); @@ -92,14 +84,9 @@ void DevelopmentToolDockingWindow::dispose() maObjectInspectorTreeHandler.dispose(); // dispose welded objects - mpClassNameLabel.reset(); - mpInterfacesTreeView.reset(); - mpServicesTreeView.reset(); - mpPropertiesTreeView.reset(); - mpMethodsTreeView.reset(); + mpObjectInspectorWidgets.reset(); mpSelectionToggle.reset(); mpDocumentModelTreeView.reset(); - mpObjectInspectorToolbar.reset(); SfxDockingWindow::dispose(); } diff --git a/sfx2/source/devtools/ObjectInspectorTreeHandler.cxx b/sfx2/source/devtools/ObjectInspectorTreeHandler.cxx index 6dd8c40ab9ad..b7f1391ff17f 100644 --- a/sfx2/source/devtools/ObjectInspectorTreeHandler.cxx +++ b/sfx2/source/devtools/ObjectInspectorTreeHandler.cxx @@ -777,52 +777,44 @@ uno::Reference<uno::XInterface> getSelectedXInterface(weld::TreeView const& rTre } // end anonymous namespace ObjectInspectorTreeHandler::ObjectInspectorTreeHandler( - std::unique_ptr<weld::TreeView>& pInterfacesTreeView, - std::unique_ptr<weld::TreeView>& pServicesTreeView, - std::unique_ptr<weld::TreeView>& pPropertiesTreeView, - std::unique_ptr<weld::TreeView>& pMethodsTreeView, - std::unique_ptr<weld::Label>& pClassNameLabel, - std::unique_ptr<weld::Toolbar>& pObjectInspectorToolbar, - std::unique_ptr<weld::Notebook>& pObjectInspectorNotebook) - : mpInterfacesTreeView(pInterfacesTreeView) - , mpServicesTreeView(pServicesTreeView) - , mpPropertiesTreeView(pPropertiesTreeView) - , mpMethodsTreeView(pMethodsTreeView) - , mpClassNameLabel(pClassNameLabel) - , mpObjectInspectorToolbar(pObjectInspectorToolbar) - , mpObjectInspectorNotebook(pObjectInspectorNotebook) + std::unique_ptr<ObjectInspectorWidgets>& pObjectInspectorWidgets) + : mpObjectInspectorWidgets(pObjectInspectorWidgets) , mxContext(comphelper::getProcessComponentContext()) { - mpInterfacesTreeView->connect_expanding( + mpObjectInspectorWidgets->mpInterfacesTreeView->connect_expanding( LINK(this, ObjectInspectorTreeHandler, ExpandingHandlerInterfaces)); - mpServicesTreeView->connect_expanding( + mpObjectInspectorWidgets->mpServicesTreeView->connect_expanding( LINK(this, ObjectInspectorTreeHandler, ExpandingHandlerServices)); - mpPropertiesTreeView->connect_expanding( + mpObjectInspectorWidgets->mpPropertiesTreeView->connect_expanding( LINK(this, ObjectInspectorTreeHandler, ExpandingHandlerProperties)); - mpMethodsTreeView->connect_expanding( + mpObjectInspectorWidgets->mpMethodsTreeView->connect_expanding( LINK(this, ObjectInspectorTreeHandler, ExpandingHandlerMethods)); - mpPropertiesTreeView->connect_popup_menu( + mpObjectInspectorWidgets->mpPropertiesTreeView->connect_popup_menu( LINK(this, ObjectInspectorTreeHandler, PopupMenuHandler)); - mpInterfacesTreeView->connect_changed(LINK(this, ObjectInspectorTreeHandler, SelectionChanged)); - mpServicesTreeView->connect_changed(LINK(this, ObjectInspectorTreeHandler, SelectionChanged)); - mpPropertiesTreeView->connect_changed(LINK(this, ObjectInspectorTreeHandler, SelectionChanged)); - mpMethodsTreeView->connect_changed(LINK(this, ObjectInspectorTreeHandler, SelectionChanged)); - - mpInterfacesTreeView->make_sorted(); - mpServicesTreeView->make_sorted(); - mpPropertiesTreeView->make_sorted(); - mpMethodsTreeView->make_sorted(); - - mpObjectInspectorToolbar->connect_clicked( + mpObjectInspectorWidgets->mpInterfacesTreeView->connect_changed( + LINK(this, ObjectInspectorTreeHandler, SelectionChanged)); + mpObjectInspectorWidgets->mpServicesTreeView->connect_changed( + LINK(this, ObjectInspectorTreeHandler, SelectionChanged)); + mpObjectInspectorWidgets->mpPropertiesTreeView->connect_changed( + LINK(this, ObjectInspectorTreeHandler, SelectionChanged)); + mpObjectInspectorWidgets->mpMethodsTreeView->connect_changed( + LINK(this, ObjectInspectorTreeHandler, SelectionChanged)); + + mpObjectInspectorWidgets->mpInterfacesTreeView->make_sorted(); + mpObjectInspectorWidgets->mpServicesTreeView->make_sorted(); + mpObjectInspectorWidgets->mpPropertiesTreeView->make_sorted(); + mpObjectInspectorWidgets->mpMethodsTreeView->make_sorted(); + + mpObjectInspectorWidgets->mpToolbar->connect_clicked( LINK(this, ObjectInspectorTreeHandler, ToolbarButtonClicked)); - mpObjectInspectorToolbar->set_item_sensitive("inspect", false); - mpObjectInspectorToolbar->set_item_sensitive("back", false); + mpObjectInspectorWidgets->mpToolbar->set_item_sensitive("inspect", false); + mpObjectInspectorWidgets->mpToolbar->set_item_sensitive("back", false); - mpObjectInspectorNotebook->connect_leave_page( + mpObjectInspectorWidgets->mpNotebook->connect_leave_page( LINK(this, ObjectInspectorTreeHandler, NotebookLeavePage)); - mpObjectInspectorNotebook->connect_enter_page( + mpObjectInspectorWidgets->mpNotebook->connect_enter_page( LINK(this, ObjectInspectorTreeHandler, NotebookEnterPage)); } @@ -841,27 +833,27 @@ void ObjectInspectorTreeHandler::handleExpanding(std::unique_ptr<weld::TreeView> IMPL_LINK(ObjectInspectorTreeHandler, ExpandingHandlerInterfaces, weld::TreeIter const&, rParent, bool) { - handleExpanding(mpInterfacesTreeView, rParent); + handleExpanding(mpObjectInspectorWidgets->mpInterfacesTreeView, rParent); return true; } IMPL_LINK(ObjectInspectorTreeHandler, ExpandingHandlerServices, weld::TreeIter const&, rParent, bool) { - handleExpanding(mpServicesTreeView, rParent); + handleExpanding(mpObjectInspectorWidgets->mpServicesTreeView, rParent); return true; } IMPL_LINK(ObjectInspectorTreeHandler, ExpandingHandlerProperties, weld::TreeIter const&, rParent, bool) { - handleExpanding(mpPropertiesTreeView, rParent); + handleExpanding(mpObjectInspectorWidgets->mpPropertiesTreeView, rParent); return true; } IMPL_LINK(ObjectInspectorTreeHandler, ExpandingHandlerMethods, weld::TreeIter const&, rParent, bool) { - handleExpanding(mpMethodsTreeView, rParent); + handleExpanding(mpObjectInspectorWidgets->mpMethodsTreeView, rParent); return true; } @@ -869,7 +861,7 @@ IMPL_LINK(ObjectInspectorTreeHandler, SelectionChanged, weld::TreeView&, rTreeVi { bool bHaveNodeWithObject = false; - if (mpPropertiesTreeView.get() == &rTreeView) + if (mpObjectInspectorWidgets->mpPropertiesTreeView.get() == &rTreeView) { auto* pNode = getSelectedNode(rTreeView); if (auto* pBasicValueNode = dynamic_cast<BasicValueNode*>(pNode)) @@ -880,7 +872,7 @@ IMPL_LINK(ObjectInspectorTreeHandler, SelectionChanged, weld::TreeView&, rTreeVi } } - mpObjectInspectorToolbar->set_item_sensitive("inspect", bHaveNodeWithObject); + mpObjectInspectorWidgets->mpToolbar->set_item_sensitive("inspect", bHaveNodeWithObject); } IMPL_LINK(ObjectInspectorTreeHandler, PopupMenuHandler, const CommandEvent&, rCommandEvent, bool) @@ -888,15 +880,15 @@ IMPL_LINK(ObjectInspectorTreeHandler, PopupMenuHandler, const CommandEvent&, rCo if (rCommandEvent.GetCommand() != CommandEventId::ContextMenu) return false; - auto xInterface = getSelectedXInterface(*mpPropertiesTreeView); + auto xInterface = getSelectedXInterface(*mpObjectInspectorWidgets->mpPropertiesTreeView); if (xInterface.is()) { - std::unique_ptr<weld::Builder> xBuilder( - Application::CreateBuilder(mpPropertiesTreeView.get(), "sfx/ui/devtoolsmenu.ui")); + std::unique_ptr<weld::Builder> xBuilder(Application::CreateBuilder( + mpObjectInspectorWidgets->mpPropertiesTreeView.get(), "sfx/ui/devtoolsmenu.ui")); std::unique_ptr<weld::Menu> xMenu(xBuilder->weld_menu("inspect_menu")); OString sCommand( - xMenu->popup_at_rect(mpPropertiesTreeView.get(), + xMenu->popup_at_rect(mpObjectInspectorWidgets->mpPropertiesTreeView.get(), tools::Rectangle(rCommandEvent.GetMousePosPixel(), Size(1, 1)))); if (sCommand == "inspect") @@ -912,7 +904,7 @@ IMPL_LINK(ObjectInspectorTreeHandler, ToolbarButtonClicked, const OString&, rSel { if (rSelectionId == "inspect") { - auto xInterface = getSelectedXInterface(*mpPropertiesTreeView); + auto xInterface = getSelectedXInterface(*mpObjectInspectorWidgets->mpPropertiesTreeView); if (xInterface.is()) { addToStack(uno::Any(xInterface)); @@ -930,7 +922,7 @@ IMPL_LINK(ObjectInspectorTreeHandler, ToolbarButtonClicked, const OString&, rSel } else if (rSelectionId == "refresh") { - auto rPageId = mpObjectInspectorNotebook->get_current_page_ident(); + auto rPageId = mpObjectInspectorWidgets->mpNotebook->get_current_page_ident(); NotebookEnterPage(rPageId); } } @@ -943,31 +935,31 @@ IMPL_LINK(ObjectInspectorTreeHandler, NotebookEnterPage, const OString&, rPageId uno::Reference<uno::XInterface> xInterface(aAny, uno::UNO_QUERY); if (rPageId == "object_inspector_interfaces_tab") { - mpInterfacesTreeView->freeze(); - clearAll(mpInterfacesTreeView); + mpObjectInspectorWidgets->mpInterfacesTreeView->freeze(); + clearAll(mpObjectInspectorWidgets->mpInterfacesTreeView); appendInterfaces(xInterface); - mpInterfacesTreeView->thaw(); + mpObjectInspectorWidgets->mpInterfacesTreeView->thaw(); } else if (rPageId == "object_inspector_services_tab") { - mpServicesTreeView->freeze(); - clearAll(mpServicesTreeView); + mpObjectInspectorWidgets->mpServicesTreeView->freeze(); + clearAll(mpObjectInspectorWidgets->mpServicesTreeView); appendServices(xInterface); - mpServicesTreeView->thaw(); + mpObjectInspectorWidgets->mpServicesTreeView->thaw(); } else if (rPageId == "object_inspector_properties_tab") { - mpPropertiesTreeView->freeze(); - clearAll(mpPropertiesTreeView); + mpObjectInspectorWidgets->mpPropertiesTreeView->freeze(); + clearAll(mpObjectInspectorWidgets->mpPropertiesTreeView); appendProperties(xInterface); - mpPropertiesTreeView->thaw(); + mpObjectInspectorWidgets->mpPropertiesTreeView->thaw(); } else if (rPageId == "object_inspector_methods_tab") { - mpMethodsTreeView->freeze(); - clearAll(mpMethodsTreeView); + mpObjectInspectorWidgets->mpMethodsTreeView->freeze(); + clearAll(mpObjectInspectorWidgets->mpMethodsTreeView); appendMethods(xInterface); - mpMethodsTreeView->thaw(); + mpObjectInspectorWidgets->mpMethodsTreeView->thaw(); } } } @@ -976,27 +968,27 @@ IMPL_LINK(ObjectInspectorTreeHandler, NotebookLeavePage, const OString&, rPageId { if (rPageId == "object_inspector_interfaces_tab") { - mpInterfacesTreeView->freeze(); - clearAll(mpInterfacesTreeView); - mpInterfacesTreeView->thaw(); + mpObjectInspectorWidgets->mpInterfacesTreeView->freeze(); + clearAll(mpObjectInspectorWidgets->mpInterfacesTreeView); + mpObjectInspectorWidgets->mpInterfacesTreeView->thaw(); } else if (rPageId == "object_inspector_services_tab") { - mpServicesTreeView->freeze(); - clearAll(mpServicesTreeView); - mpServicesTreeView->thaw(); + mpObjectInspectorWidgets->mpServicesTreeView->freeze(); + clearAll(mpObjectInspectorWidgets->mpServicesTreeView); + mpObjectInspectorWidgets->mpServicesTreeView->thaw(); } else if (rPageId == "object_inspector_properties_tab") { - mpPropertiesTreeView->freeze(); - clearAll(mpPropertiesTreeView); - mpPropertiesTreeView->thaw(); + mpObjectInspectorWidgets->mpPropertiesTreeView->freeze(); + clearAll(mpObjectInspectorWidgets->mpPropertiesTreeView); + mpObjectInspectorWidgets->mpPropertiesTreeView->thaw(); } else if (rPageId == "object_inspector_methods_tab") { - mpMethodsTreeView->freeze(); - clearAll(mpMethodsTreeView); - mpMethodsTreeView->thaw(); + mpObjectInspectorWidgets->mpMethodsTreeView->freeze(); + clearAll(mpObjectInspectorWidgets->mpMethodsTreeView); + mpObjectInspectorWidgets->mpMethodsTreeView->thaw(); } return true; } @@ -1050,7 +1042,7 @@ void ObjectInspectorTreeHandler::appendInterfaces(uno::Reference<uno::XInterface for (auto const& xType : xSequenceTypes) { auto xClass = convertTypeToIdlClass(xType, mxContext); - lclAppendNode(mpInterfacesTreeView, new ClassNode(xClass)); + lclAppendNode(mpObjectInspectorWidgets->mpInterfacesTreeView, new ClassNode(xClass)); } } } @@ -1065,7 +1057,8 @@ void ObjectInspectorTreeHandler::appendServices(uno::Reference<uno::XInterface> const uno::Sequence<OUString> aServiceNames(xServiceInfo->getSupportedServiceNames()); for (auto const& aServiceName : aServiceNames) { - lclAppendNode(mpServicesTreeView, new SimpleStringNode(aServiceName)); + lclAppendNode(mpObjectInspectorWidgets->mpServicesTreeView, + new SimpleStringNode(aServiceName)); } } @@ -1075,7 +1068,7 @@ void ObjectInspectorTreeHandler::appendProperties(uno::Reference<uno::XInterface if (!xInterface.is()) return; GenericPropertiesNode aNode("", uno::Any(xInterface), "", mxContext); - aNode.fillChildren(mpPropertiesTreeView, nullptr); + aNode.fillChildren(mpObjectInspectorWidgets->mpPropertiesTreeView, nullptr); } /** Append methods to the "methods" tree view */ @@ -1090,13 +1083,13 @@ void ObjectInspectorTreeHandler::appendMethods(uno::Reference<uno::XInterface> c const auto xMethods = xIntrospectionAccess->getMethods(beans::MethodConcept::ALL); for (auto const& xMethod : xMethods) { - lclAppendNode(mpMethodsTreeView, new MethodNode(xMethod)); + lclAppendNode(mpObjectInspectorWidgets->mpMethodsTreeView, new MethodNode(xMethod)); } } void ObjectInspectorTreeHandler::updateBackButtonState() { - mpObjectInspectorToolbar->set_item_sensitive("back", maInspectionStack.size() > 1); + mpObjectInspectorWidgets->mpToolbar->set_item_sensitive("back", maInspectionStack.size() > 1); } // Clears all the objects from the stack @@ -1131,10 +1124,10 @@ void ObjectInspectorTreeHandler::inspectObject(uno::Reference<uno::XInterface> c // Set implementation name auto xServiceInfo = uno::Reference<lang::XServiceInfo>(xInterface, uno::UNO_QUERY); OUString aImplementationName = xServiceInfo->getImplementationName(); - mpClassNameLabel->set_label(aImplementationName); + mpObjectInspectorWidgets->mpClassNameLabel->set_label(aImplementationName); // Fire entering the current opened page manually - auto rPageId = mpObjectInspectorNotebook->get_current_page_ident(); + auto rPageId = mpObjectInspectorWidgets->mpNotebook->get_current_page_ident(); NotebookEnterPage(rPageId); } @@ -1151,10 +1144,10 @@ void ObjectInspectorTreeHandler::introspect(uno::Reference<uno::XInterface> cons void ObjectInspectorTreeHandler::dispose() { // We need to clear all the nodes - clearAll(mpInterfacesTreeView); - clearAll(mpServicesTreeView); - clearAll(mpPropertiesTreeView); - clearAll(mpMethodsTreeView); + clearAll(mpObjectInspectorWidgets->mpInterfacesTreeView); + clearAll(mpObjectInspectorWidgets->mpServicesTreeView); + clearAll(mpObjectInspectorWidgets->mpPropertiesTreeView); + clearAll(mpObjectInspectorWidgets->mpMethodsTreeView); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits