Unified Diff: lib/url.js
Issue 29371763:
Issue 4795 - Use modern JavaScript syntax (Closed)
Patch Set: Workaround limiation with iterating element collections in older versions of Chrome
Created Jan. 16, 2017, 8:46 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« lib/notificationHelper.js ('K')
|
« lib/uninstall.js ('k')
|
lib/utils.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/url.js |
| diff --git a/lib/url.js b/lib/url.js |
| index 4ea741dbbfee3134f421426713b877edf6f39364..d821f9a9ac297dc33344d5249dc5fcca5f82ff83 100644 |
| --- a/lib/url.js |
| +++ b/lib/url.js |
| @@ -17,6 +17,8 @@ |
| /** @module url */ |
| +"use strict"; |
| + |
| let getDomain = require("tldjs").getDomain; |
| let punycode = require("punycode"); |
| @@ -28,7 +30,7 @@ let getDecodedHostname = |
| * @return {string} |
| * @static |
| */ |
| -exports.getDecodedHostname = function(url) |
| +exports.getDecodedHostname = url => |
| { |
| let hostname = url.hostname; |
| @@ -46,7 +48,7 @@ exports.getDecodedHostname = function(url) |
| * @param {Frame} frame |
| * @return {string} |
| */ |
| -exports.extractHostFromFrame = function(frame) |
| +exports.extractHostFromFrame = frame => |
| { |
| for (; frame; frame = frame.parent) |
| { |
| @@ -65,7 +67,7 @@ exports.extractHostFromFrame = function(frame) |
| * @param {URL} url |
| * @return {string} |
| */ |
| -exports.stringifyURL = function(url) |
| +exports.stringifyURL = url => |
| { |
| let protocol = url.protocol; |
| let href = url.href; |
| @@ -102,7 +104,7 @@ function isDomain(hostname) |
| * @param {string} documentHost The IDN-decoded hostname of the document |
| * @return {Boolean} |
| */ |
| -exports.isThirdParty = function(url, documentHost) |
| +exports.isThirdParty = (url, documentHost) => |
| { |
| let requestHost = getDecodedHostname(url).replace(/\.+$/, ""); |
| documentHost = documentHost.replace(/\.+$/, ""); |
« lib/notificationHelper.js ('K')
|
« lib/uninstall.js ('k')
|
lib/utils.js » ('j')
|
no next file with comments »
