framework: avoid double-free in ToolBarManager()

The timer could run after the ToolBarManager is deleted because the
dispose() method can accidentally restart the timer somehow.

Change-Id: I86868f08f436976761e814d4cea1cd5a0e348935

framework/source/uielement/toolbarmanager.cxx

@@ -476,7 +476,9 @@ throw ( RuntimeException, std::exception )
 {
     SolarMutexGuard g;
     if ( Action.Action == FrameAction_CONTEXT_CHANGED )
+    {
         m_aAsyncUpdateControllersTimer.Start();
+    }
 }
 
 void SAL_CALL ToolBarManager::statusChanged( const ::com::sun::star::frame::FeatureStateEvent& Event )
@@ -1414,7 +1416,9 @@ void ToolBarManager::FillToolbar( const Reference< XIndexAccess >& rItemContaine
     if( m_pToolBar->WillUsePopupMode() )
         UpdateControllers();
     else if ( m_pToolBar->IsReallyVisible() )
+    {
         m_aAsyncUpdateControllersTimer.Start();
+    }
 
     // Try to retrieve UIName from the container property set and set it as the title
     // if it is not empty.
@@ -2048,7 +2052,9 @@ IMPL_LINK( ToolBarManager, StateChanged, StateChangedType*, pStateChangedType )
     else if ( *pStateChangedType == StateChangedType::VISIBLE )
     {
         if ( m_pToolBar->IsReallyVisible() )
+        {
             m_aAsyncUpdateControllersTimer.Start();
+        }
     }
     else if ( *pStateChangedType == StateChangedType::INITSHOW )
     {