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

Issue 2956633002: Policy implementation for encryptfs to ext4 migration strategy (Closed)

Created:
3 years, 6 months ago by igorcov
Modified:
3 years, 6 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3112
Project:
chromium
Visibility:
Public.

Description

Policy implementation for encryptfs to ext4 migration strategy Chrome OS encrypts each user’s data with the user’s password. With M60, Chrome OS is switching from ecryptfs to ext4 encryption. ARC N requires ext4 encryption. The policy DeviceEcryptfsMigrationStrategy states whether the device is allowed to use ARC or not. If it's not allowed, the process of data migration should not be launched. Design doc: https://docs.google.com/document/d/18Nz2OClU6CDb7TpZ2MbO4yySvgiOYWsFUxrR4LQYzXU/edit?ts=5915e354# This implementation is planned to become obsolete, and be rolled back after the migration process will finish. The issue for that: http://crbug.com/725493 BUG=722371 TEST=Manual tests and unit tests. Review-Url: https://codereview.chromium.org/2890843002 Cr-Original-Commit-Position: refs/heads/master@{#481499} Review-Url: https://codereview.chromium.org/2956633002 . Cr-Commit-Position: refs/branch-heads/3112@{#448} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} Committed: https://chromium.googlesource.com/chromium/src/+/f3deed295ff4ee4d8e0a42a4d9bb19d528dd2d2e

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+224 lines, -2 lines) Patch
M chrome/browser/chromeos/arc/arc_session_manager.cc View 1 chunk +8 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/arc/arc_util.h View 1 chunk +12 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/arc/arc_util.cc View 5 chunks +71 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/arc/arc_util_unittest.cc View 3 chunks +59 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/login/existing_user_controller.cc View 2 chunks +6 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/login/screens/user_selection_screen.cc View 2 chunks +2 lines, -1 line 0 comments Download
M chrome/browser/chromeos/policy/device_policy_decoder_chromeos.cc View 1 chunk +11 lines, -0 lines 0 comments Download
M chrome/test/data/policy/policy_test_cases.json View 1 chunk +5 lines, -0 lines 0 comments Download
M chromeos/chromeos_switches.h View 1 chunk +1 line, -0 lines 0 comments Download
M chromeos/chromeos_switches.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/policy/resources/policy_templates.json View 2 chunks +42 lines, -1 line 0 comments Download
M tools/metrics/histograms/enums.xml View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
igorcov
3 years, 6 months ago (2017-06-23 11:19:59 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
f3deed295ff4ee4d8e0a42a4d9bb19d528dd2d2e.

Powered by Google App Engine
This is Rietveld 408576698