Unified Diff: lib/url.js
Issue 29374674:
Issue 4864 - Start using ESLint for adblockpluschrome (Closed)
Patch Set: Tidy up hasRecord and some notification logic
Created March 31, 2017, 3:36 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« lib/devtools.js ('K')
|
« lib/tldjs.js ('k')
|
lib/utils.js » ('j')
|
notification.js » ('J')
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 63782e88a550f0cd4b66a23e606f19ab0c00db2c..36dcf66ad62c24925ddb557a12c5dbe7ea3a817a 100644 |
| --- a/lib/url.js |
| +++ b/lib/url.js |
| @@ -32,7 +32,7 @@ let getDecodedHostname = |
| */ |
| exports.getDecodedHostname = url => |
| { |
| - let hostname = url.hostname; |
| + let {hostname} = url; |
| if (hostname.indexOf("xn--") == -1) |
| return hostname; |
| @@ -69,12 +69,11 @@ exports.extractHostFromFrame = frame => |
| */ |
| exports.stringifyURL = url => |
| { |
| - let protocol = url.protocol; |
| - let href = url.href; |
| + let {protocol, href} = url; |
| if (protocol == "http:" || protocol == "https:") |
| { |
| - let hostname = url.hostname; |
| + let {hostname} = url; |
| if (hostname.indexOf("xn--") != -1) |
| href = href.replace(hostname, punycode.toUnicode(hostname)); |
