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

Unified Diff: lib/url.js

Issue 6318719470927872: Noissue - Fixed documentation where parameters where mistakently indicated as optional (Closed)
Patch Set: Fixed typo Created March 2, 2015, 5:20 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/icon.js ('k') | no next file » | 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
@@ -49,7 +49,7 @@
/**
* Gets the IDN-decoded hostname from a URL object.
*
- * @param {URL} [url]
+ * @param {URL} url
* @return {string}
*/
function getDecodedHostname(url)
@@ -68,7 +68,7 @@
* If the URL don't have host information (like "about:blank"
* and "data:" URLs) it falls back to the parent frame.
*
- * @param {Frame} [frame]
+ * @param {Frame} frame
* @return {string}
*/
function extractHostFromFrame(frame)
@@ -88,7 +88,7 @@
* Converts a URL object into a string. For HTTP(S) URLs
* the hostname gets IDN-decoded and the hash is stripped.
*
- * @param {URL} [url]
+ * @param {URL} url
* @return {string}
*/
function stringifyURL(url)
@@ -148,8 +148,8 @@
/**
* Checks whether the request's origin is different from the document's origin.
*
- * @param {URL} [url] The request URL
- * @param {string} [documentHost] The IDN-decoded hostname of the document
+ * @param {URL} url The request URL
+ * @param {string} documentHost The IDN-decoded hostname of the document
* @return {Boolean}
*/
function isThirdParty(url, documentHost)
« no previous file with comments | « lib/icon.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld