Resolves: tdf#144343 This isn't used to toggle navigator on/off

but instead contains a SfxBoolItem which specifies if it should
be shown or hidden and ChildWindowExecute takes care of deciding
if that requires Toggling on or off.

Change-Id: I2ad422041bb6f40f5b726111cf5ce35a32c39128
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/193310
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Tested-by: Jenkins

svx/source/form/fmshell.cxx

@@ -597,7 +597,9 @@ void FmFormShell::Execute(SfxRequest &rReq)
         case SID_FM_FILTER_NAVIGATOR:
         case SID_FM_SHOW_DATANAVIGATOR :
         {
-            GetViewShell()->GetViewFrame().ToggleChildWindow(nSlot);
+            // request may explicitly request to show/hide these
+            // so may or may not result in a ToggleChildWindow
+            GetViewShell()->GetViewFrame().ChildWindowExecute(rReq);
             rReq.Done();
         }   break;
         case SID_FM_SHOW_FMEXPLORER: