Unified Diff: lib/icon.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.
« no previous file with comments
|
« no previous file
|
lib/url.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/icon.js |
| =================================================================== |
| --- a/lib/icon.js |
| +++ b/lib/icon.js |
| @@ -88,8 +88,8 @@ |
| * Set the browser action icon for the given page, indicating whether |
| * adblocking is active there, and considering the icon animation. |
| * |
| - * @param {Page} [page] The page to set the browser action icon for |
| - * @param {Boolean} [whitelisted] Whether the page has been whitelisted |
| + * @param {Page} page The page to set the browser action icon for |
| + * @param {Boolean} whitelisted Whether the page has been whitelisted |
| */ |
| function updateIcon(page, whitelisted) |
| { |
| @@ -101,7 +101,7 @@ |
| /** |
| * Starts to animate the browser action icon to indicate a pending notifcation. |
| * |
| - * @param {string} [type] The notifaction type (i.e: "information" or "critical") |
| + * @param {string} type The notification type (i.e: "information" or "critical") |
| */ |
| function startIconAnimation(type) |
| { |
« no previous file with comments
|
« no previous file
|
lib/url.js » ('j')
|
no next file with comments »
