Context menu tracking when Skinning

            // 12/22/25: RDH PR 11388058. This code was changed to call the frame window so the menu displays a Dark
            // theme. That api routes through Codejock's CXTPCommandBars::TrackPopupMenu, which calls
            // XTPMouseManager::SendTrackLost on entry. SendTrackLost walks CXTPMouseManager::m_arrTracked calling
            // OnTrackLost, which ends tracking mode on the MRUPopupBar and closes the vertical MRU out from under us.
            //
            // 8/31/26 RDH: That was originally worked around by passing TPM_RECURSE, because Codejock skips
            // SendTrackLost when that flag is set. But TPM_RECURSE does not mean "leave my popup alone" - it also
            // calls LockTrackRecurse(TRUE), which bumps m_nLockRecurse on EVERY bar already being tracked, and
            // m_nLockRecurse is the flag that gates all hot tracking (CXTPPopupBar::OnMouseMove,
            // CXTPCommandBar::OnMouseMove and OnMouseLeave, the hover timers, CXTPMouseManager::DeliverMessage).
            // The result was a context menu whose items never highlighted, and stale highlights left lit on the MRU
            // items underneath because nothing was allowed to clear them.
            //
            // Removing just our own bar from the tracked array is not enough, and it is worth knowing why. An
            // MRUPopupBar is a CXTPRibbonSystemPopupBarPage, that is, a page of the system menu popup that spawned
            // it, and that parent bar is tracked too. SendTrackLost calls OnTrackLost on the parent, the parent
            // closes its child popups, and ours goes down with it however carefully we detached it. That is exactly
            // what TPM_RECURSE was papering over by locking the whole array rather than one entry.
            //
            // So do what TPM_RECURSE did, minus the freeze: lift the entire tracked array out for the duration of
            // the call and put it back afterwards. SendTrackLost then has nothing to walk and is a no op, every
            // bar keeps its tracking mode, and because nobody gets m_nLockRecurse set the context menu hot tracks
            // normally. The bars are restored in their original order, since SetTrack appends.
            //
            //
            // Do not try to solve this by vetoing dismissal in MRUPopupBar::SetTrackingMode instead. SendTrackLost
            // is "while (m_arrTracked.GetSize() > 0) m_arrTracked[0]->OnTrackLost();" - a veto that stops a bar
            // being removed from the array hangs the thread.
            CXTPCommandBar*   pOwningBar     = DYNAMIC_DOWNCAST(MRUPopupBar, GetParent());
            CXTPMouseManager* pMouseManager  = XTPMouseManager();

            std::vector<CXTPCommandBar*> vecSuspendedBars;

            if(pOwningBar && pMouseManager)
            {
                CXTPMouseManager::CTrackArray& arrTracked = pMouseManager->GetTrackArray();

                for(int i = 0; i < (int)arrTracked.GetSize(); i++)
                {
                    vecSuspendedBars.push_back(arrTracked[i]);
                }

                for(size_t i = 0; i < vecSuspendedBars.size(); i++)
                {
                    pMouseManager->RemoveTrack(vecSuspendedBars[i]);
                }
            }

            BOOL item = pFrameWnd->TrackPopupMenu(subMenu, TPM_RETURNCMD | TPM_LEFTALIGN | TPM_LEFTBUTTON, point.x, point.y, this, nullptr, JNone, hInstance);

            for(size_t i = 0; i < vecSuspendedBars.size(); i++)
            {
                // The window can be gone if a command tore the bar down while the menu was up.
                if(::IsWindow(vecSuspendedBars[i]->GetSafeHwnd()))
                {
                    pMouseManager->SetTrack(vecSuspendedBars[i], FALSE);
                }
            }