[Pkg-mozext-commits] [requestpolicy] 85/257: [fix] Don't use the return value of `Cu.import()`

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository requestpolicy.

commit 00b5521415c4ebbeddf841493f0094e27f62aaae
Author: Martin Kimmerle <dev at 256k.de>
Date:   Fri Oct 16 01:26:49 2015 +0200

    [fix] Don't use the return value of `Cu.import()`
---
 src/content/ui/frame.js | 29 ++++++++++++++++++-----------
 1 file changed, 18 insertions(+), 11 deletions(-)

diff --git a/src/content/ui/frame.js b/src/content/ui/frame.js
index cbfe8af..05d0cca 100644
--- a/src/content/ui/frame.js
+++ b/src/content/ui/frame.js
@@ -39,17 +39,24 @@ Components.utils.import("resource://gre/modules/devtools/Console.jsm");
   const Cu = Components.utils;
 
   // import some modules
-  let {ScriptLoader} = Cu.import(
-      "chrome://rpcontinued/content/lib/script-loader.jsm", {});
-  let mod = {};
-  ScriptLoader.importModules([
-    "lib/utils/constants",
-    "lib/logger",
-    "lib/environment",
-    "lib/framescript-to-overlay-communication"
-  ], mod);
-  let {C, Logger, Environment, FrameScriptEnvironment,
-       FramescriptToOverlayCommunication} = mod;
+  let {
+    ScriptLoader,
+    C,
+    Logger,
+    Environment,
+    FrameScriptEnvironment,
+    FramescriptToOverlayCommunication
+  } = (function () {
+    let mod = {};
+    Cu.import("chrome://rpcontinued/content/lib/script-loader.jsm", mod);
+    mod.ScriptLoader.importModules([
+      "lib/utils/constants",
+      "lib/logger",
+      "lib/environment",
+      "lib/framescript-to-overlay-communication"
+    ], mod);
+    return mod;
+  })();
 
 
   let framescriptEnv = new FrameScriptEnvironment(mm);

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/requestpolicy.git