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

Issue 2922913002: cros: Prevent shut down on lid-close during migration. (Closed)

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

Description

cros: Prevent shut down on lid-close during migration. In the login screen, the system will shut down on lid close. Although we ask users not to close the lid during the migration, it should be better to change the lid-close action to SUSPEND during migration. BUG=726229 TEST=out/Release/chromeos_unittests --gtest_filter=PowerPolicyControllerTest.* Review-Url: https://codereview.chromium.org/2912763003 Cr-Original-Commit-Position: refs/heads/master@{#476091} Review-Url: https://codereview.chromium.org/2922913002 . Cr-Commit-Position: refs/branch-heads/3112@{#138} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} Committed: https://chromium.googlesource.com/chromium/src/+/f4f285abd6c1b70b7ea16e6ed03f68e56cbc5df6

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+63 lines, -4 lines) Patch
M chrome/browser/ui/webui/chromeos/login/encryption_migration_screen_handler.cc View 2 chunks +7 lines, -3 lines 0 comments Download
M chromeos/dbus/power_policy_controller.h View 2 chunks +9 lines, -0 lines 0 comments Download
M chromeos/dbus/power_policy_controller.cc View 2 chunks +19 lines, -1 line 0 comments Download
M chromeos/dbus/power_policy_controller_unittest.cc View 1 chunk +28 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (1 generated)
kinaba
merging...
3 years, 6 months ago (2017-06-05 02:38:53 UTC) #1
kinaba
3 years, 6 months ago (2017-06-05 02:40:09 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
f4f285abd6c1b70b7ea16e6ed03f68e56cbc5df6.

Powered by Google App Engine
This is Rietveld 408576698