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

Unified Diff: lib/url.js

Issue 29371763: Issue 4795 - Use modern JavaScript syntax (Closed)
Patch Set: Undo accidental whitespace change Created Jan. 16, 2017, 2:57 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
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(/\.+$/, "");

Powered by Google App Engine
This is Rietveld