jitsi/jitsi — pull request #28 by cobratbq (2014-05-03 21:13:05+00:00)
Title: Only create the chat room if provider cannot find it.

When MUCServiceImpl creates a chat room (i.e. a ChatRoomWrapper instance) it should first check to see if the chat room already exists and is already known by the protocol.

By first looking for an existing chat room, findRoom(...), before creating the chat room, we can make sure that we are not redundantly creating a chat room (and causing an OperationFailedException as a consequence).
Also, this becomes particularly relevant when the IRC protocol implementation is merged, since the IRC servers may automatically join a user to a particular channel, in which case the ChatRoom instance (related to the protocol implementation) exists, but the ChatRoomWrapper (related to the Jitsi core framework) does not yet.