Chromium Code Reviews| Index: ui/webui/resources/js/local_strings.js |
| diff --git a/ui/webui/resources/js/local_strings.js b/ui/webui/resources/js/local_strings.js |
| index 3209612e2ff518f9e7cd5ce93b2240bcdba7d853..0b72b5c49e958885267291fc1ab6b5e497525e04 100644 |
| --- a/ui/webui/resources/js/local_strings.js |
| +++ b/ui/webui/resources/js/local_strings.js |
| @@ -9,7 +9,7 @@ |
| * {@code templateData}. This class provides a simpler interface to access those |
| * strings. |
| * |
| - * @param {Object} opt_templateData Optional object containing translated |
| + * @param {Object=} opt_templateData Optional object containing translated |
|
arv (Not doing code reviews)
2014/07/24 16:44:24
No need to include "Optional" in the description s
Dan Beam
2014/07/25 01:52:33
Done.
|
| * strings. If this is not supplied during construction, it can be |
| * assigned to the templateData property after construction. If all else |
| * fails, the value of window.templateDate will be used. |
| @@ -26,7 +26,8 @@ function LocalStrings(opt_templateData) { |
| * Returns a formatted string where $1 to $9 are replaced by the second to the |
| * tenth argument. |
| * @param {string} s The format string. |
| - * @param {...string} The extra values to include in the formatted output. |
| + * @param {Arguments} args The extra values to include in the formatted |
| + * output. |
| * @return {string} The string after format substitution. |
| */ |
| function replaceArgs(s, args) { |
| @@ -49,13 +50,13 @@ function trimAccelerators(s) { |
| LocalStrings.prototype = { |
| /** |
| * The template data object. |
| - * @type {Object} |
| + * @type {Object|undefined} |
| */ |
| - templateData: null, |
| + templateData: undefined, |
| /** |
| * Gets a localized string by its id. |
| - * @param {string} s The ID of the string we want. |
| + * @param {string} id The ID of the string we want. |
| * @return {string} The localized string. |
| */ |
| getString: function(id) { |
| @@ -72,7 +73,8 @@ LocalStrings.prototype = { |
| * Returns a formatted localized string where $1 to $9 are replaced by the |
| * second to the tenth argument. |
| * @param {string} id The ID of the string we want. |
| - * @param {...string} The extra values to include in the formatted output. |
| + * @param {...string} var_args The extra values to include in the formatted |
| + * output. |
| * @return {string} The formatted string. |
| */ |
| getStringF: function(id, var_args) { |