New issue
Advanced search Search tips

Issue 769646 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 612328



Sign in to add a comment

Device Service: Eliminate //device/base

Project Member Reported by blundell@chromium.org, Sep 28 2017

Issue description

As part of completing the Device Service and eliminating //device, we need to eliminate //device/base. This work will be best done as an end stage of the Device Service work (i.e., when all the device features have been servicified). At that point, it should be relatively straightforward:

- DeviceClient will presumably be a dead interface that can be eliminated.
- It looks like DeviceInfoQueryWin and DeviceMonitorWin are used only in //device, so they can just be moved into //services/device to be part of the Device Service internal implementation.
- DeviceMonitorLinux is owned by //content's browser_main_loop but it looks like it's used only within //device. We will need to change it to be owned by the Device Service and then it can be moved to be part of the Device Service internal implementation.
- //device/base/synchronization is shared by //device feature impls and their clients. It can become a client library of the Device Service.

Further analysis will be required for DeviceInfoQueryWin, DeviceMonitorWin, DeviceMonitorLinux to make sure that they're not somehow used in some subtle way from outside //device.
 
Components: Internals>Services>Device
Project Member

Comment 2 by sheriffbot@chromium.org, Nov 7

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment