Not in the onAdded handler.

Otherwise it gets debounced for multiple instances and certain chat boxes will
then not get shown.

Don't call trimChats when the box is minimized or maximized, instead call it
only when the minimized chats toggle appears or disappears, since that's the
element that adds/removes space.

Also, a fix. 'onShow' has been renamed to 'onMaximized'

updates #443

updates #443

In the process refactored the code being tested.

- Move notifications code in MUC to src/converse-notification.js
- Trigger the 'contactStatusChanged' event in RosterContact and not in
  ChatBoxView (which might not exist).

prevous fix wasn't correct. Rename handlers to make it clear which methods are
the event handlers, put them together and make sure docstrings are relevant.

The no-jquery build was broken

According to the require.js documentation, if you are using "require" in HTML,
then the data-main attribute should not be used.

http://www.requirejs.org/docs/api.html#data-main

So merged main.js into converse.js, which appears to solve the problem.

Method has new arg signature now that it's an event handler

Corrected spelling, punctuation and terminology.

Don't make the event "this" context the protected converse object.
Can't trigger multiple data parameters, need to pass an object if there are
multiple values to be sent.

Also add a config setting for the icon shown in HTML5 notificatins.

updates #443

So that they are retrievable via the API.

updates #443

They're independent of the controlbox and used by other core functions.

- Refactored sound notification to make it a bit more generic
- Split up onMessage

Still have to move the roster code out of src/converse-controlbox.js