Don't reset the view controls state just because we are setting the already active tool state again

(cherry picked from commit 132ecee665)

Co-authored-by: Marek Roszko <mark.roszko@gmail.com>

common/tool/tool_manager.cpp

@@ -1229,6 +1229,9 @@ bool TOOL_MANAGER::processEvent( const TOOL_EVENT& aEvent )
 
 void TOOL_MANAGER::setActiveState( TOOL_STATE* aState )
 {
+    if( m_activeState == aState )
+        return;
+
     if( m_activeState && m_viewControls )
         saveViewControls( m_activeState );