Revert "world map: use game state change event to load quest icons"

This reverts commit 97c484fc38.

runelite-client/src/main/java/net/runelite/client/callback/ClientThread.java

@@ -110,10 +110,6 @@ public class ClientThread
 			{
 				ir.remove();
 			}
-			else
-			{
-				log.trace("Deferring task {}", r);
-			}
 		}
 	}
 }

runelite-client/src/main/java/net/runelite/client/plugins/worldmap/WorldMapPlugin.java

@@ -34,7 +34,6 @@ import net.runelite.api.Client;
 import net.runelite.api.GameState;
 import net.runelite.api.Quest;
 import net.runelite.api.Skill;
-import net.runelite.api.events.GameStateChanged;
 import net.runelite.api.events.StatChanged;
 import net.runelite.api.events.WidgetLoaded;
 import net.runelite.api.widgets.WidgetID;
@@ -169,15 +168,6 @@ public class WorldMapPlugin extends Plugin
 		woodcuttingLevel = 0;
 	}
 
-	@Subscribe
-	public void onGameStateChanged(GameStateChanged gameStateChanged)
-	{
-		if (gameStateChanged.getGameState() == GameState.LOGGED_IN)
-		{
-			updateQuestStartPointIcons();
-		}
-	}
-
 	@Subscribe
 	public void onConfigChanged(ConfigChanged event)
 	{
@@ -498,14 +488,17 @@ public class WorldMapPlugin extends Plugin
 		}
 
 		// Must setup the quest icons on the client thread, after the player has logged in.
-		clientThread.invoke(() ->
+		clientThread.invokeLater(() ->
 		{
-			if (client.getGameState() == GameState.LOGGED_IN)
+			if (client.getGameState() != GameState.LOGGED_IN)
 			{
-				Arrays.stream(QuestStartLocation.values())
+				return false;
-					.map(this::createQuestStartPoint)
-					.forEach(worldMapPointManager::add);
 			}
+
+			Arrays.stream(QuestStartLocation.values())
+				.map(this::createQuestStartPoint)
+				.forEach(worldMapPointManager::add);
+			return true;
 		});
 	}