commit 066cf8849fb760e12609578eb474c4d73659a63e
author Benjamin Prosnitz <bprosnitz@google.com> Mon Jun 22 13:37:50 2015 -0700
committer Benjamin Prosnitz <bprosnitz@google.com> Mon Jun 22 13:38:20 2015 -0700
tree fed236c3d1632525f20c7ecf54864469edb29188
parent 2ea68ff95cdbf460f2949c8655ec8aeb026efb9b

ext: Stop NACL plugin when unused

Change-Id: I51cb007fdbdcd8e2464e7af96962e7c7c66c1881

extension/src/background/index.js

diff --git a/extension/src/background/index.js b/extension/src/background/index.js
index 3e055de..15aee31 100644
--- a/extension/src/background/index.js
+++ b/extension/src/background/index.js
@@ -160,8 +160,12 @@
     console.log('Cleaned up instance: ' + instanceId + ' in ' +
                 (end - now) + ' ms');
     bp.instanceIds.set(port, _.remove(bp.instanceIds.get(port), [instanceId]));
+    if (bp.instanceIds.get(port).length === 0) {
+      bp.instanceIds.delete(port);
+    }
     delete bp.ports[instanceId];
     sendCleanupFinishedMessage();
+    bp.stopNaclPluginIfUnused();
   });
 };
 
@@ -256,9 +260,15 @@
   bp.nacl.once('ready', cb.bind(bp));
 };
 
+// Stop the nacl plugin if it is not currently used.
+BackgroundPage.prototype.stopNaclPluginIfUnused = function() {
+  if (Object.keys(this.ports).length === 0) {
+    this.stopNaclPlugin();
+  }
+};
+
 // Stop the nacl plug-in - remove it from the page and clean up state.
 BackgroundPage.prototype.stopNaclPlugin = function() {
-  // TODO(bprosnitz) Should we call nacl.cleanupInstance()?
   this.nacl.destroy();
   delete this.nacl;
 };