Rework list handling in ResourceManager by having NetworkClient own its own list rather than sharing ResourceManager's list

DetectionManager.cpp

@@ -1392,8 +1392,6 @@ void DetectionManager::ProcessCleanup()
 {
     WaitForDetection();
 
-    SignalUpdate(DETECTIONMANAGER_UPDATE_REASON_RGBCONTROLLER_LIST_CLEARED);
-
     /*-----------------------------------------------------*\
     | Make a copy of the list so that the controllers can   |
     | be deleted after the list is cleared                  |
@@ -1405,6 +1403,11 @@ void DetectionManager::ProcessCleanup()
     \*-----------------------------------------------------*/
     rgb_controllers.clear();
 
+    /*-----------------------------------------------------*\
+    | Signal the list cleared callback                      |
+    \*-----------------------------------------------------*/
+    SignalUpdate(DETECTIONMANAGER_UPDATE_REASON_RGBCONTROLLER_LIST_CLEARED);
+
     /*-----------------------------------------------------*\
     | Delete the controllers                                |
     \*-----------------------------------------------------*/