diff --git a/zanata-war/src/main/java/org/zanata/webtrans/client/service/UserSessionService.java b/zanata-war/src/main/java/org/zanata/webtrans/client/service/UserSessionService.java
index 8e6cac91cd..0dfeb8f138 100644
--- a/zanata-war/src/main/java/org/zanata/webtrans/client/service/UserSessionService.java
+++ b/zanata-war/src/main/java/org/zanata/webtrans/client/service/UserSessionService.java
@@ -93,6 +93,7 @@ public void addUser(EditorClientId editorClientId, UserPanelSessionItem item)
    public void removeUser(EditorClientId editorClientId)
    {
       userSessionMap.remove(editorClientId);
+      // FIXME remove from colorListMap (memory leak)
    }
 
    public Map<EditorClientId, UserPanelSessionItem> getUserSessionMap()
@@ -100,6 +101,8 @@ public Map<EditorClientId, UserPanelSessionItem> getUserSessionMap()
       return userSessionMap;
    }
 
+   // TODO what we pass is really editorClientId
+   // Should we be passing sessionId?  See also memory leak above.
    public String getColor(String sessionId)
    {
       if (colorListMap.containsKey(sessionId))