[Libreoffice-commits] core.git: include/sfx2 sfx2/source

Szymon Kłos eszkadev at gmail.com
Thu Mar 16 15:45:57 UTC 2017


 include/sfx2/notebookbar/NotebookbarTabControl.hxx |    3 ++-
 sfx2/source/notebookbar/NotebookbarTabControl.cxx  |   15 +++++++--------
 2 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit c97d78eaa95e9ff0cef7b467ba566af56e3e7c27
Author: Szymon Kłos <eszkadev at gmail.com>
Date:   Thu Mar 16 09:15:55 2017 +0100

    tdf#103355 Working menu after closing slideshow
    
    Change-Id: Ic6bfb6f9a86d9c475d2d3368b924b56a89c90551
    Reviewed-on: https://gerrit.libreoffice.org/35251
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Szymon Kłos <eszkadev at gmail.com>

diff --git a/include/sfx2/notebookbar/NotebookbarTabControl.hxx b/include/sfx2/notebookbar/NotebookbarTabControl.hxx
index 062fe9631f48..0d4ce9b9632c 100644
--- a/include/sfx2/notebookbar/NotebookbarTabControl.hxx
+++ b/include/sfx2/notebookbar/NotebookbarTabControl.hxx
@@ -32,9 +32,10 @@ private:
                                           const OUString& aModuleName,
                                           ToolBox* pShortcuts
                              );
-    DECL_STATIC_LINK(NotebookbarTabControl, OpenNotebookbarPopupMenu, NotebookBar*, void);
+    DECL_LINK(OpenNotebookbarPopupMenu, NotebookBar*, void);
 
     ChangedUIEventListener* m_pListener;
+    css::uno::Reference<css::frame::XFrame> m_xFrame;
 
 protected:
     bool m_bInitialized;
diff --git a/sfx2/source/notebookbar/NotebookbarTabControl.cxx b/sfx2/source/notebookbar/NotebookbarTabControl.cxx
index 47a230097542..a443c045ffa1 100644
--- a/sfx2/source/notebookbar/NotebookbarTabControl.cxx
+++ b/sfx2/source/notebookbar/NotebookbarTabControl.cxx
@@ -143,7 +143,7 @@ void NotebookbarTabControl::StateChanged(StateChangedType nStateChange)
         pShortcuts->Show();
 
         SetToolBox( static_cast<ToolBox*>( pShortcuts.get() ) );
-        SetIconClickHdl( LINK( nullptr, NotebookbarTabControl, OpenNotebookbarPopupMenu ) );
+        SetIconClickHdl( LINK( this, NotebookbarTabControl, OpenNotebookbarPopupMenu ) );
 
         m_pListener = new ChangedUIEventListener( this );
 
@@ -159,10 +159,10 @@ void NotebookbarTabControl::StateChanged(StateChangedType nStateChange)
 
         Reference<XComponentContext> xContext = comphelper::getProcessComponentContext();
         const Reference<XModuleManager> xModuleManager  = ModuleManager::create( xContext );
-        Reference<XFrame> xFrame = SfxViewFrame::Current()->GetFrame().GetFrameInterface();
-        OUString aModuleName = xModuleManager->identify( xFrame );
+        m_xFrame = SfxViewFrame::Current()->GetFrame().GetFrameInterface();
+        OUString aModuleName = xModuleManager->identify( m_xFrame );
 
-        FillShortcutsToolBox( xContext, xFrame, aModuleName, pToolBox );
+        FillShortcutsToolBox( xContext, m_xFrame, aModuleName, pToolBox );
 
         Size aSize( pToolBox->GetOptimalSize() );
         Point aPos( ICON_SIZE + 10, 0 );
@@ -229,14 +229,13 @@ void NotebookbarTabControl::FillShortcutsToolBox(Reference<XComponentContext>& x
     }
 }
 
-IMPL_STATIC_LINK(NotebookbarTabControl, OpenNotebookbarPopupMenu, NotebookBar*, pNotebookbar, void)
+IMPL_LINK(NotebookbarTabControl, OpenNotebookbarPopupMenu, NotebookBar*, pNotebookbar, void)
 {
-    SfxViewFrame* pViewFrame = SfxViewFrame::Current();
-    if (pNotebookbar && pViewFrame)
+    if (pNotebookbar && m_xFrame.is())
     {
         Sequence<Any> aArgs {
             makeAny(comphelper::makePropertyValue("Value", OUString("notebookbar"))),
-            makeAny(comphelper::makePropertyValue("Frame", pViewFrame->GetFrame().GetFrameInterface())) };
+            makeAny(comphelper::makePropertyValue("Frame", m_xFrame)) };
 
         Reference<XComponentContext> xContext = comphelper::getProcessComponentContext();
         Reference<XPopupMenuController> xPopupController(


More information about the Libreoffice-commits mailing list