Two things...

1) The determining if the chatRoom is visible or not should probably be done up in the manager, not here.  When the manager gets a message, he passes the message to the ChatRoom to update its view, and then the manager ensures that chatroom is focused.

2) The check for the if could be another method on the HiddenChatRoomViewController.  Maybe there's a contains() method that takes a name and checks if that name is contained within the indicator.  Otherwise, things outside the controller would have to know how the controller is actually working.  If we changed it to not use a dropdown or a list later, we have to update code outside of that controller.