New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 730952 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Jun 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 689410



Sign in to add a comment

Convert chrome/browser/ui/webui/chromeos/login/encryption_migration_screen_handler.cc from using PowerSaveBlocker to using Wake Lock Mojo interface

Project Member Reported by ke...@intel.com, Jun 8 2017

Issue description

See parent bug.
 

Comment 1 by ke...@intel.com, Jun 8 2017

Blocking: 689410
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d2b4e2b209ebbf5a61fbb97f8590c267990e2a72

commit d2b4e2b209ebbf5a61fbb97f8590c267990e2a72
Author: Ke He <ke.he@intel.com>
Date: Wed Jun 14 05:26:07 2017

Convert EncryptionMigrationScreenHandler to be client of wake lock

Wake Lock is a Mojo interface that wraps PowerSaveBlocker. As part of the
creation of standalone Device Service, all browser-side clients of
PowerSaveBlocker should be converted to be clients of the Wake Lock Mojo
interface instead.

BUG= 730952 

Change-Id: I07f9a843b08fca413073e90e8dc96e7e3ddaa990
Reviewed-on: https://chromium-review.googlesource.com/527938
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Commit-Queue: Ke He <ke.he@intel.com>
Cr-Commit-Position: refs/heads/master@{#479294}
[modify] https://crrev.com/d2b4e2b209ebbf5a61fbb97f8590c267990e2a72/chrome/browser/ui/BUILD.gn
[modify] https://crrev.com/d2b4e2b209ebbf5a61fbb97f8590c267990e2a72/chrome/browser/ui/webui/chromeos/DEPS
[modify] https://crrev.com/d2b4e2b209ebbf5a61fbb97f8590c267990e2a72/chrome/browser/ui/webui/chromeos/login/encryption_migration_screen_handler.cc
[modify] https://crrev.com/d2b4e2b209ebbf5a61fbb97f8590c267990e2a72/chrome/browser/ui/webui/chromeos/login/encryption_migration_screen_handler.h

Comment 3 by ke...@intel.com, Jun 26 2017

Status: Fixed (was: Started)

Sign in to add a comment