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

Unified Diff: chrome/browser/resources/media_router/media_router_data.js

Issue 1998013003: [Media Router WebUI] Show issue's secondary action when 'dismiss'. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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: chrome/browser/resources/media_router/media_router_data.js
diff --git a/chrome/browser/resources/media_router/media_router_data.js b/chrome/browser/resources/media_router/media_router_data.js
index b33f0b941e100bcf4d16c07cb94232f3e27db272..5465d8649fab9842d3895d548d681fefee02b7d2 100644
--- a/chrome/browser/resources/media_router/media_router_data.js
+++ b/chrome/browser/resources/media_router/media_router_data.js
@@ -119,7 +119,7 @@ cr.define('media_router', function() {
* @param {string} title The issue title.
* @param {string} message The issue message.
* @param {number} defaultActionType The type of default action.
- * @param {?number} secondaryActionType The type of optional action.
+ * @param {number|undefined} secondaryActionType The type of optional action.
* @param {?string} routeId The route ID to which this issue
* pertains. If not set, this is a global issue.
* @param {boolean} isBlocking True if this issue blocks other UI.
@@ -142,7 +142,7 @@ cr.define('media_router', function() {
/** @type {number} */
this.defaultActionType = defaultActionType;
- /** @type {?number} */
+ /** @type {number|undefined} */
this.secondaryActionType = secondaryActionType;
/** @type {?string} */

Powered by Google App Engine
This is Rietveld 408576698