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

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

Issue 2819613003: cros: Update design of encryption migration UI for ext4 dircrypto. (Closed)
Patch Set: Address review comments. Created 3 years, 8 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/screen_encryption_migration.js
diff --git a/chrome/browser/resources/chromeos/login/screen_encryption_migration.js b/chrome/browser/resources/chromeos/login/screen_encryption_migration.js
index 200ede461ec2bc80f2eeaa7d24f3eeb3aa65ff8a..70740d3483ee3d495879d83203c29d1ee8d5b71a 100644
--- a/chrome/browser/resources/chromeos/login/screen_encryption_migration.js
+++ b/chrome/browser/resources/chromeos/login/screen_encryption_migration.js
@@ -9,7 +9,9 @@ login.createScreen('EncryptionMigrationScreen', 'encryption-migration',
'setUIState',
'setMigrationProgress',
'setIsResuming',
- 'setBatteryPercent',
+ 'setBatteryState',
+ 'setAvailableSpaceInString',
+ 'setNecessarySpaceInString',
],
/**
@@ -73,11 +75,29 @@ login.createScreen('EncryptionMigrationScreen', 'encryption-migration',
* Updates battery level of the device.
* @param {number} batteryPercent Battery level in percent.
* @param {boolean} isEnoughBattery True if the battery is enough.
+ * @param {boolena} isCharging True if the device is connected to power.
*/
- setBatteryPercent: function(batteryPercent, isEnoughBattery) {
- $('encryption-migration-element').batteryPercent =
- Math.floor(batteryPercent);
- $('encryption-migration-element').isEnoughBattery = isEnoughBattery;
+ setBatteryState: function(batteryPercent, isEnoughBattery, isCharging) {
+ var element = $('encryption-migration-element');
+ element.batteryPercent = Math.floor(batteryPercent);
+ element.isEnoughBattery = isEnoughBattery;
+ element.isCharging = isCharging;
+ },
+
+ /**
+ * Updates the string representation of available space size.
+ * @param {string} space
+ */
+ setAvailableSpaceInString: function(space) {
+ $('encryption-migration-element').availableSpaceInString = space;
+ },
+
+ /**
+ * Updates the string representation of necessary space size.
+ * @param {string} space
+ */
+ setNecessarySpaceInString: function(space) {
+ $('encryption-migration-element').necessarySpaceInString = space;
},
};
});

Powered by Google App Engine
This is Rietveld 408576698