Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/crawler.js

Issue 29355247: Noissue - fix according to coding style (Closed)
Patch Set: Created Sept. 29, 2016, 8:46 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: lib/crawler.js
diff --git a/lib/crawler.js b/lib/crawler.js
index 8e7144acee6667440528cb34e62d419bee7aa784..0eb51063d9d54c3aa51b2c0aa8d78e45a8ff0c76 100644
--- a/lib/crawler.js
+++ b/lib/crawler.js
@@ -4,13 +4,14 @@
* http://mozilla.org/MPL/2.0/.
*/
+'use strict';
+
/**
* @module crawler
*/
-Cu.import("resource://gre/modules/Services.jsm");
-Cu.import("resource://gre/modules/Task.jsm");
-Cu.import("resource://gre/modules/Promise.jsm");
+const {XPCOMUtils} = Cu.import("resource://gre/modules/XPCOMUtils.jsm", {});
+const {Task} = Cu.import("resource://gre/modules/Task.jsm");
Wladimir Palant 2016/09/29 10:36:10 Please add the second parameter, otherwise it stil
sergei 2016/09/29 12:46:16 Done in https://codereview.adblockplus.org/2935527
function abprequire(module)
{

Powered by Google App Engine
This is Rietveld