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

Unified Diff: chrome/browser/resources/chromeos/arc_support/background.js

Issue 2104893003: Adds a new checkbox for backup and restore in opt-in flow. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge with ToT Created 4 years, 6 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/chromeos/arc_support/background.js
diff --git a/chrome/browser/resources/chromeos/arc_support/background.js b/chrome/browser/resources/chromeos/arc_support/background.js
index 84a1d4de92ac44548f34cc42a93d8abf6145d910..87088f00507824dba35f09fbab5f2111cb1b4c8a 100644
--- a/chrome/browser/resources/chromeos/arc_support/background.js
+++ b/chrome/browser/resources/chromeos/arc_support/background.js
@@ -106,6 +106,7 @@ function initialize(data, deviceId, silentMode) {
loadTimeData.data = data;
appWindow.contentWindow.i18nTemplate.process(doc, loadTimeData);
var countryCode = data.countryCode.toLowerCase();
+ setBackupRestoreMode(data.textBackupRestore);
var scriptSetCountryCode = 'document.countryCode = \'' + countryCode + '\';';
termsView.addContentScripts([
@@ -126,6 +127,17 @@ function initialize(data, deviceId, silentMode) {
}
/**
+ * Handles the event when the user clicks on a learn more link. Opens the
+ * support page for the user.
+ * @param {Event} event
+ */
+var onLearnMore = function(event) {
+ var url = 'https://support.google.com/chromebook?p=playapps';
+ chrome.browser.openTab({'url': url}, function() {});
+ event.preventDefault();
+};
+
+/**
* Sets current metrics mode.
* @param {string} text Describes current metrics state.
* @param {boolean} canEnable Defines if user is allowed to change this metrics
@@ -143,21 +155,30 @@ function setMetricsMode(text, canEnable, on) {
event.preventDefault();
};
- var onLearnMore = function(event) {
- var url = 'https://support.google.com/chromebook?p=playapps';
- chrome.browser.openTab({'url': url}, function() {});
- event.preventDefault();
- };
-
doc.getElementById('text-metrics').innerHTML = text;
doc.getElementById('settings-link').addEventListener('click', onSettings);
- doc.getElementById('learn-more-link').addEventListener('click', onLearnMore);
+ doc.getElementById('learn-more-link-metrics').addEventListener('click',
+ onLearnMore);
// Applying metrics mode changes page layout, update terms height.
updateTermsHeight();
}
/**
+ * Sets current metrics mode.
+ * @param {string} text String used to display next to checkbox.
+ */
+function setBackupRestoreMode(text) {
+ var doc = appWindow.contentWindow.document;
+ var enableBackupRestoreEl = doc.getElementById('enable-backup-restore');
+ enableBackupRestoreEl.checked = true;
xiyuan 2016/06/29 21:36:38 We should take the on/off value passed in from Arc
malaykeshav 2016/06/30 02:46:19 The default value if always checked.
+
+ doc.getElementById('text-backup-restore').innerHTML = text;
+ doc.getElementById('learn-more-link-backup-restore').addEventListener('click',
+ onLearnMore);
+}
+
+/**
* Updates terms view height manually because webview is not automatically
* resized in case parent div element gets resized.
*/
@@ -415,6 +436,12 @@ chrome.app.runtime.onLaunched.addListener(function() {
if (!enableMetrics.hidden && enableMetrics.checked) {
sendNativeMessage('enableMetrics');
}
+
+ var enableBackupRestore = doc.getElementById('enable-backup-restore');
+ if (enableBackupRestore.checked) {
+ sendNativeMessage('enableBackupRestore');
+ }
+
sendNativeMessage('startLso');
};

Powered by Google App Engine
This is Rietveld 408576698