Move roster view code into a separate plugin

converse.js

@@ -46,6 +46,7 @@ require.config({
         // Converse
         "converse-api":             "src/converse-api",
         "converse-chatview":        "src/converse-chatview",
+        "converse-rosterview":      "src/converse-rosterview",
         "converse-controlbox":      "src/converse-controlbox",
         "converse-core":            "src/converse-core",
         "converse-headline":        "src/converse-headline",

src/converse-core.js

@@ -761,17 +761,18 @@
             }.bind(this), 200));
         };
 
+        this.afterReconnected = function () {
+            this.chatboxes.registerMessageHandler();
+            this.xmppstatus.sendPresence();
+            this.giveFeedback(__('Contacts'));
+        };
+
         this.onReconnected = function () {
             // We need to re-register all the event handlers on the newly
             // created connection.
             var deferred = new $.Deferred();
             this.initStatus(function () {
-                // FIXME: leaky abstraction from RosterView
-                this.rosterview.registerRosterXHandler();
-                this.rosterview.registerPresenceHandler();
-                this.chatboxes.registerMessageHandler();
-                this.xmppstatus.sendPresence();
-                this.giveFeedback(__('Contacts'));
+                this.afterReconnected();
                 deferred.resolve();
             }.bind(this));
             return deferred.promise();

src/converse-notification.js

@@ -36,7 +36,6 @@
                 notification_icon: '/logo/conversejs.png'
             });
 
-
             converse.isOnlyChatStateNotification = function ($msg) {
                 // See XEP-0085 Chat State Notification
                 return (