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

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

Issue 1582943003: [Media Router] Add cloud services preferences. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 11 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_ui_interface.js
diff --git a/chrome/browser/resources/media_router/media_router_ui_interface.js b/chrome/browser/resources/media_router/media_router_ui_interface.js
index 8c869a9facf26edf2c8e04247e28179d87c5acf6..2f467b6712cb5fca086f64a13dd4d333ea30188e 100644
--- a/chrome/browser/resources/media_router/media_router_ui_interface.js
+++ b/chrome/browser/resources/media_router/media_router_ui_interface.js
@@ -55,7 +55,8 @@ cr.define('media_router.ui', function() {
* sinks: !Array<!media_router.Sink>,
* routes: !Array<!media_router.Route>,
* castModes: !Array<!media_router.CastMode>,
- * wasFirstRunFlowAcknowledged: boolean}} data
+ * wasFirstRunFlowAcknowledged: boolean,
+ * showFirstRunFlowCloudPref: boolean}} data
* Parameters in data:
* firstRunFlowCloudPrefLearnMoreUrl - url to open when the cloud services
* pref learn more link is clicked.
@@ -65,6 +66,8 @@ cr.define('media_router.ui', function() {
* castModes - list of available cast modes.
* wasFirstRunFlowAcknowledged - true if first run flow was previously
* acknowledged by user.
+ * showFirstRunFlowCloudPref - true if the cloud pref option should be
+ * shown.
*/
function setInitialData(data) {
container.firstRunFlowCloudPrefLearnMoreUrl =
@@ -74,6 +77,8 @@ cr.define('media_router.ui', function() {
container.allSinks = data['sinks'];
container.routeList = data['routes'];
container.showFirstRunFlow = !data['wasFirstRunFlowAcknowledged'];
+ container.showFirstRunFlowCloudPref =
+ data['showFirstRunFlowCloudPref'];
container.maybeShowRouteDetailsOnOpen();
media_router.browserApi.onInitialDataReceived();
}
@@ -134,9 +139,12 @@ cr.define('media_router.browserApi', function() {
/**
* Indicates that the user has acknowledged the first run flow.
+ *
+ * @param {boolean} optedIntoCloudServices Whether or not the user opted into
+ * cloud services.
*/
- function acknowledgeFirstRunFlow() {
- chrome.send('acknowledgeFirstRunFlow');
+ function acknowledgeFirstRunFlow(optedIntoCloudServices) {
+ chrome.send('acknowledgeFirstRunFlow', [optedIntoCloudServices]);
}
/**

Powered by Google App Engine
This is Rietveld 408576698