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

Unified Diff: chrome/browser/resources/chromeos/login/oobe_screen_eula.js

Issue 755203002: Added usage of ScreenContext in EulaScreen. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase, fix to Screen.initialize(). Created 6 years, 1 month 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/login/oobe_screen_eula.js
diff --git a/chrome/browser/resources/chromeos/login/oobe_screen_eula.js b/chrome/browser/resources/chromeos/login/oobe_screen_eula.js
index 7cafec281cb52b082a7ffa508255b1adf0eb6ef3..686023f8ae2817134f054d8aa4f893b01550d334 100644
--- a/chrome/browser/resources/chromeos/login/oobe_screen_eula.js
+++ b/chrome/browser/resources/chromeos/login/oobe_screen_eula.js
@@ -7,6 +7,8 @@
*/
login.createScreen('EulaScreen', 'eula', function() {
+ var CONTEXT_KEY_USAGE_STATS_ENABLED = 'usageStatsEnabled';
+
return {
/** @override */
decorate: function() {
@@ -34,6 +36,14 @@ login.createScreen('EulaScreen', 'eula', function() {
}, 0);
event.preventDefault();
});
+
+ var self = this;
+ $('usage-stats').addEventListener('click', function(event) {
+ self.context.set(CONTEXT_KEY_USAGE_STATS_ENABLED,
+ $('usage-stats').checked);
+ self.commitContextChanges();
+ event.stopPropagation();
+ });
},
/**
@@ -79,8 +89,10 @@ login.createScreen('EulaScreen', 'eula', function() {
var backButton = this.ownerDocument.createElement('button');
backButton.id = 'back-button';
backButton.textContent = loadTimeData.getString('back');
+
+ var self = this;
backButton.addEventListener('click', function(e) {
- chrome.send('eulaOnExit', [false, $('usage-stats').checked]);
+ chrome.send('eulaBackButtonClicked');
e.stopPropagation();
});
buttons.push(backButton);
@@ -92,7 +104,7 @@ login.createScreen('EulaScreen', 'eula', function() {
acceptButton.textContent = loadTimeData.getString('acceptAgreement');
acceptButton.addEventListener('click', function(e) {
$('eula').classList.add('loading'); // Mark EULA screen busy.
- chrome.send('eulaOnExit', [true, $('usage-stats').checked]);
+ chrome.send('eulaAcceptButtonClicked');
e.stopPropagation();
});
buttons.push(acceptButton);
« no previous file with comments | « chrome/browser/resources/chromeos/login/oobe-screen.js ('k') | chrome/browser/ui/webui/chromeos/login/base_screen_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698