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

Unified Diff: lib/url.js

Issue 29452181: Noissue - Merge current tip to Edge bookmark (Closed)
Patch Set: Created May 30, 2017, 3:49 p.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
« no previous file with comments | « lib/uninstall.js ('k') | lib/utils.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/url.js
===================================================================
--- a/lib/url.js
+++ b/lib/url.js
@@ -1,6 +1,6 @@
/*
* This file is part of Adblock Plus <https://adblockplus.org/>,
- * Copyright (C) 2006-2016 Eyeo GmbH
+ * Copyright (C) 2006-2017 eyeo GmbH
*
* Adblock Plus is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 3 as
@@ -32,7 +32,7 @@
*/
exports.getDecodedHostname = url =>
{
- let hostname = url.hostname;
+ let {hostname} = url;
if (hostname.indexOf("xn--") == -1)
return hostname;
@@ -69,12 +69,11 @@
*/
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));
« no previous file with comments | « lib/uninstall.js ('k') | lib/utils.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld