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

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

Issue 904163003: Reset Screen moved to ScreenContext. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: User actions introduced. Created 5 years, 10 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/login/oobe_screen_reset_confirmation_overlay.js
diff --git a/chrome/browser/resources/chromeos/login/oobe_screen_reset_confirmation_overlay.js b/chrome/browser/resources/chromeos/login/oobe_screen_reset_confirmation_overlay.js
index 188acb9e7585dcd62ed7e40299bcb28231af4982..05715a5a12135de0d866114373f054b2b4a7962a 100644
--- a/chrome/browser/resources/chromeos/login/oobe_screen_reset_confirmation_overlay.js
+++ b/chrome/browser/resources/chromeos/login/oobe_screen_reset_confirmation_overlay.js
@@ -4,6 +4,8 @@
cr.define('reset', function() {
+ var USER_ACTION_POWERWASH_PRESSED = 'powerwash-pressed';
+ var CONTEXT_KEY_IS_CONFIRMATIONAL_VIEW = 'is-confirmational-view';
/**
* ResetScreenConfirmationOverlay class
* Encapsulated handling of the 'Confirm reset device' overlay OOBE page.
@@ -33,7 +35,8 @@ cr.define('reset', function() {
* @param {Event} e The click event.
*/
handleDismiss_: function(e) {
- $('reset').isConfirmational = false;
+ $('reset').context.set(CONTEXT_KEY_IS_CONFIRMATIONAL_VIEW, false);
+ $('reset').commitContextChanges();
$('overlay-reset').setAttribute('hidden', true);
e.stopPropagation();
},
@@ -43,8 +46,10 @@ cr.define('reset', function() {
* @param {Event} e The click event.
*/
handleCommit_: function(e) {
- $('reset').isConfirmational = false;
- chrome.send('powerwashOnReset', [$('reset').rollbackChecked]);
+ $('reset').context.set(CONTEXT_KEY_IS_CONFIRMATIONAL_VIEW, false);
+ $('reset').commitContextChanges();
+ $('reset').send(login.Screen.CALLBACK_USER_ACTED,
+ USER_ACTION_POWERWASH_PRESSED);
$('overlay-reset').setAttribute('hidden', true);
e.stopPropagation();
},

Powered by Google App Engine
This is Rietveld 408576698