[UNTESTED] Remove unnecessary commas for ie8 compatibility

The new changes made to strophe.roster.js are incompatible with the way
converse.js works.

Will likely replace strophe.roster.js completely.

Instead, just populate the roster from sessionStorage if available.

Seems like the RID gets incremented once too many (outside of upper bound of
expected window), and then server responds with item-not-found.

See: http://xmpp.org/extensions/xep-0124.html

called sequentially with enough time between each call.

Index must always be 0 if there aren't any group elements in the dom yet.

which checks if a contact should appear in the roster (depends on
show_only_online_users setting)

Don't sort the global ContactRosters collection (we only need to sort the
individual groups).
Only add the roster to the DOM once the users have been added.

In the previous commit, the mock connection object was refactored to use a real
Strophe.Connection object.

This caused a test in spec/controlbox.js to fail (due to a method that was now
no longer mocked).

Added a quick workaround (via monkeypatch) for now.

Also update documentation to mention msginit.

No use in using a passed in connection that's not connected.

Also fixed groupchat test by getting and including correct msg id.

That way, if need be, we could sort messages.