Improve Server.connect so child services trigger Server port onConnect.

darkwallet · Apr 8, 2014 · 8618a6d · 8618a6d
1 parent 7144802
commit 8618a6d
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/js/backend/services.js b/js/backend/services.js
@@ -1,5 +1,6 @@
 define(function () {
     var allPorts = {};
+    var instances = {};
     var Services = {
         /*
          * Start the given service
@@ -12,6 +13,11 @@ define(function () {
             if (!allPorts.hasOwnProperty(name)) {
                 allPorts[name] = [];
             }
+            if (!instances.hasOwnProperty(name)) {
+                instances[name] = {onConnect: onConnect, onDisconnect: onDisconnect};
+            } else {
+                throw Error("Service with duplicate name!");
+            }
             chrome.runtime.onConnect.addListener(function(port) {
               if (port.name == name) {
                 allPorts[name].push(port)
@@ -48,7 +54,11 @@ define(function () {
             if (!allPorts.hasOwnProperty(name)) {
                 allPorts[name] = [];
             }
-            allPorts[name].push({postMessage: onMessage});
+            var port = {postMessage: onMessage};
+            allPorts[name].push(port);
+            console.log("["+name+"] connect child service");
+            instances[name].onConnect ? instances[name].onConnect(port) : null;
+            return port;
         }
     };
     return Services;