Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(343)

Unified Diff: ui/webui/resources/js/cr/ui/alert_overlay.js

Issue 475633006: Typecheck JS files for chrome://extensions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@D_asserts_codingconvention
Patch Set: looked twice Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/webui/resources/js/cr/ui/alert_overlay.js
diff --git a/ui/webui/resources/js/cr/ui/alert_overlay.js b/ui/webui/resources/js/cr/ui/alert_overlay.js
index 76ec5014d39d521eb6880cd82d850361a1abb954..4bfd081606ab98f30d419dcf34597fb3d80bd7ce 100644
--- a/ui/webui/resources/js/cr/ui/alert_overlay.js
+++ b/ui/webui/resources/js/cr/ui/alert_overlay.js
@@ -42,17 +42,18 @@ cr.define('alertOverlay', function() {
* callbacks.
* @param {string} title The alert title to display to the user.
* @param {string} message The alert message to display to the user.
- * @param {string=} okTitle The title of the OK button. If undefined or empty,
- * no button is shown.
- * @param {string=} cancelTitle The title of the cancel button. If undefined
- * or empty, no button is shown.
- * @param {function=} okCallback A function to be called when the user presses
- * the ok button. Can be undefined if |okTitle| is falsey.
- * @param {function=} cancelCallback A function to be called when the user
- * presses the cancel button. Can be undefined if |cancelTitle| is falsey.
+ * @param {string=} opt_okTitle The title of the OK button. If undefined or
+ * empty, no button is shown.
+ * @param {string=} opt_cancelTitle The title of the cancel button. If
+ * undefined or empty, no button is shown.
+ * @param {function()=} opt_okCallback A function to be called when the user
+ * presses the ok button. Can be undefined if |opt_okTitle| is falsey.
+ * @param {function()=} opt_cancelCallback A function to be called when the
+ * user presses the cancel button. Can be undefined if |opt_cancelTitle|
+ * is falsey.
*/
- function setValues(
- title, message, okTitle, cancelTitle, okCallback, cancelCallback) {
+ function setValues(title, message, opt_okTitle, opt_cancelTitle,
+ opt_okCallback, opt_cancelCallback) {
Dan Beam 2014/08/21 18:27:48 nit: function setValues(title, message, opt_okT
Vitaly Pavlenko 2014/08/22 01:43:41 Done.
if (typeof title != 'undefined')
$('alertOverlayTitle').textContent = title;
$('alertOverlayTitle').hidden = typeof title == 'undefined';
@@ -61,15 +62,15 @@ cr.define('alertOverlay', function() {
$('alertOverlayMessage').textContent = message;
$('alertOverlayMessage').hidden = typeof message == 'undefined';
- if (okTitle)
- okButton.textContent = okTitle;
- okButton.hidden = !okTitle;
- okButton.clickCallback = okCallback;
+ if (opt_okTitle)
+ okButton.textContent = opt_okTitle;
+ okButton.hidden = !opt_okTitle;
+ okButton.clickCallback = opt_okCallback;
- if (cancelTitle)
- cancelButton.textContent = cancelTitle;
- cancelButton.hidden = !cancelTitle;
- cancelButton.clickCallback = cancelCallback;
+ if (opt_cancelTitle)
+ cancelButton.textContent = opt_cancelTitle;
+ cancelButton.hidden = !opt_cancelTitle;
+ cancelButton.clickCallback = opt_cancelCallback;
};
// Export

Powered by Google App Engine
This is Rietveld 408576698