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

Side by Side Diff: lib/crawler.js

Issue 29355271: Noissue - Don't import JavaScript modules into the global scope (includes removing unused module). (Closed)
Patch Set: Created Sept. 29, 2016, 12:05 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « lib/commandLine.js ('k') | lib/main.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * This Source Code is subject to the terms of the Mozilla Public License 2 * This Source Code is subject to the terms of the Mozilla Public License
3 * version 2.0 (the "License"). You can obtain a copy of the License at 3 * version 2.0 (the "License"). You can obtain a copy of the License at
4 * http://mozilla.org/MPL/2.0/. 4 * http://mozilla.org/MPL/2.0/.
5 */ 5 */
6 6
7 "use strict"; 7 "use strict";
8 8
9 /** 9 /**
10 * @module crawler 10 * @module crawler
11 */ 11 */
12 12
13 Cu.import("resource://gre/modules/Services.jsm"); 13 const {Services} = Cu.import("resource://gre/modules/Services.jsm", {});
14 Cu.import("resource://gre/modules/Task.jsm"); 14 const {XPCOMUtils} = Cu.import("resource://gre/modules/XPCOMUtils.jsm", {});
15 Cu.import("resource://gre/modules/Promise.jsm"); 15 const {Task} = Cu.import("resource://gre/modules/Task.jsm", {});
Wladimir Palant 2016/09/29 18:19:28 You cannot remove Promise.jsm without getting rid
sergei 2016/09/29 21:00:18 Restored. Sorry, overlooked it, thought last defe
16 16
17 function abprequire(module) 17 function abprequire(module)
18 { 18 {
19 let result = {}; 19 let result = {};
20 result.wrappedJSObject = result; 20 result.wrappedJSObject = result;
21 Services.obs.notifyObservers(result, "adblockplus-require", module); 21 Services.obs.notifyObservers(result, "adblockplus-require", module);
22 return result.exports; 22 return result.exports;
23 } 23 }
24 24
25 let {RequestNotifier} = abprequire("requestNotifier"); 25 let {RequestNotifier} = abprequire("requestNotifier");
(...skipping 370 matching lines...) Expand 10 before | Expand all | Expand 10 after
396 396
397 function reportException(e) 397 function reportException(e)
398 { 398 {
399 let stack = ""; 399 let stack = "";
400 if (e && typeof e == "object" && "stack" in e) 400 if (e && typeof e == "object" && "stack" in e)
401 stack = e.stack + "\n"; 401 stack = e.stack + "\n";
402 402
403 Cu.reportError(e); 403 Cu.reportError(e);
404 dump(e + "\n" + stack + "\n"); 404 dump(e + "\n" + stack + "\n");
405 } 405 }
OLDNEW
« no previous file with comments | « lib/commandLine.js ('k') | lib/main.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld