Resume encryption migration immediately if the previous migration is incomplete.

Resume encryption migration immediately if the previous migration is incomplete.

The previous migration can be incomplete. This can happen when, for example,
the user shut down the device during the migration.
In this case, MountEx returns a dedicated error: MOUNT_ERROR_PREVIOUS_MIGRATION_INCOMPLETE.
If Chrome get this error, the migration should start immediately without
asking the user's choice, as the user can not skip the migration.

BUG=706017
TEST=manually tested by modifying fake_cryptohome_client to return the error.

Review-Url: https://codereview.chromium.org/2801873007
Cr-Commit-Position: refs/heads/master@{#463151}
Committed: https://chromium.googlesource.com/chromium/src/+/fbd1dfec1f46320180425e602cb2751978813ecb

[fukino, 2017-04-07 11:52:44]

Description was changed from

==========
Resume encryption migration immediately if the previous migration is incomplete.

BUG=
==========

to

==========
Resume encryption migration immediately if the previous migration is incomplete.

The previous migration can be incomplete. This can happen when, for example,
the user shut down the device during the migration.
In this case, MountEx returns a dedicated error:
MOUNT_ERROR_PREVIOUS_MIGRATION_INCOMPLETE.
If Chrome get this error, the migration should start immediately without
asking the user's choice, as the user can not skip the migration.

BUG=706017
TEST=manually tested by modifying fake_cryptohome_client to return the error.
==========

[fukino, 2017-04-07 11:54:12]

fukino@chromium.org changed reviewers:
+ xiyuan@chromium.org

[fukino, 2017-04-07 11:54:12]

Xiyuan, could you take a look?

[xiyuan, 2017-04-07 15:12:17]

lgtm

[fukino, 2017-04-09 08:45:11]

On 2017/04/07 15:12:17, xiyuan wrote:
> lgtm

Thank you!

[fukino, 2017-04-09 08:45:16]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2017-04-09 08:45:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-09 21:56:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491727516015220, "parent_rev":
"dc4c6262aae691d5e6560f32ee41375be5609f29", "commit_rev":
"fbd1dfec1f46320180425e602cb2751978813ecb"}

[commit-bot: I haz the power, 2017-04-09 21:57:37]

Description was changed from

==========
Resume encryption migration immediately if the previous migration is incomplete.

The previous migration can be incomplete. This can happen when, for example,
the user shut down the device during the migration.
In this case, MountEx returns a dedicated error:
MOUNT_ERROR_PREVIOUS_MIGRATION_INCOMPLETE.
If Chrome get this error, the migration should start immediately without
asking the user's choice, as the user can not skip the migration.

BUG=706017
TEST=manually tested by modifying fake_cryptohome_client to return the error.
==========

to

==========
Resume encryption migration immediately if the previous migration is incomplete.

The previous migration can be incomplete. This can happen when, for example,
the user shut down the device during the migration.
In this case, MountEx returns a dedicated error:
MOUNT_ERROR_PREVIOUS_MIGRATION_INCOMPLETE.
If Chrome get this error, the migration should start immediately without
asking the user's choice, as the user can not skip the migration.

BUG=706017
TEST=manually tested by modifying fake_cryptohome_client to return the error.

Review-Url: https://codereview.chromium.org/2801873007
Cr-Commit-Position: refs/heads/master@{#463151}
Committed:
https://chromium.googlesource.com/chromium/src/+/fbd1dfec1f46320180425e602cb2...
==========

[commit-bot: I haz the power, 2017-04-09 21:57:38]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fbd1dfec1f46320180425e602cb2...