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

Unified Diff: chromeos/dbus/fake_cryptohome_client.cc

Issue 2818393002: cros: Mount the existing eCryptfs vault to start migration. (Closed)
Patch Set: FakeCryptohomeClient::MountEx should succeed when |to_migrate_from_ecryptfs| is true. 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: chromeos/dbus/fake_cryptohome_client.cc
diff --git a/chromeos/dbus/fake_cryptohome_client.cc b/chromeos/dbus/fake_cryptohome_client.cc
index 1410ec538be6d0a2a3219762563e98ed0071ba03..3d87897c14ed4fa974c504b7abb3a65b147c5d33 100644
--- a/chromeos/dbus/fake_cryptohome_client.cc
+++ b/chromeos/dbus/fake_cryptohome_client.cc
@@ -533,7 +533,8 @@ void FakeCryptohomeClient::MountEx(
reply.MutableExtension(cryptohome::MountReply::reply);
mount->set_sanitized_username(GetStubSanitizedUsername(cryptohome_id));
if (base::CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kTestEncryptionMigrationUI)) {
+ switches::kTestEncryptionMigrationUI) &&
+ !request.to_migrate_from_ecryptfs()) {
reply.set_error(cryptohome::CRYPTOHOME_ERROR_MOUNT_OLD_ENCRYPTION);
}
ReturnProtobufMethodCallback(reply, callback);

Powered by Google App Engine
This is Rietveld 408576698