Clarify logic: don't create global variables at all

Infocatcher · Jul 6, 2016 · 55d0ff9 · 55d0ff9
1 parent 1cb4e8d
commit 55d0ff9
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/protocol-process.js b/protocol-process.js
@@ -1,13 +1,12 @@
-Components.utils.import("resource://gre/modules/Services.jsm");
-if(Services.appinfo.processType == Services.appinfo.PROCESS_TYPE_CONTENT) {
-	Components.utils.import("chrome://privatetab/content/protocol.jsm");
+(function() {
+	var {Services} = Components.utils.import("resource://gre/modules/Services.jsm", {});
+	if(Services.appinfo.processType != Services.appinfo.PROCESS_TYPE_CONTENT)
+		return;
+	var {privateProtocol} = Components.utils.import("chrome://privatetab/content/protocol.jsm", {});
 	privateProtocol.init(function() {});
-	let global = this;
 	addMessageListener("PrivateTab:ProtocolDestroy", function destroy(msg) {
 		removeMessageListener("PrivateTab:ProtocolDestroy", destroy);
 		privateProtocol.destroy();
-		delete global.Services;
-		delete global.privateProtocol;
 		Components.utils.unload("chrome://privatetab/content/protocol.jsm");
 	});
-}
+})();