Move DeviceMonitorLinux to //device/core.

Move DeviceMonitorLinux to //device/core.

This code will be shared by HidServiceLinux and UsbServiceLinux so it
needs to be in the shared //device/core component.

Sharing the service introduces an issue with destruction order that is
resolved by adding a WillDestroyMonitorMessageLoop() method to its
Observer interface.

BUG=422562
Committed: https://crrev.com/57b6ff447084f1b0040557436d0f55c1e152626c
Cr-Commit-Position: refs/heads/master@{#396318}

[Reilly Grant (use Gerrit), 2016-05-26 18:24:25]

Fix //components/pairing build.

[Reilly Grant (use Gerrit), 2016-05-26 18:24:44]

reillyg@chromium.org changed reviewers:
+ juncai@chromium.org, zork@chromium.org

[Reilly Grant (use Gerrit), 2016-05-26 18:24:45]

juncai@, please review //device.
zork@, please review //components/pairing.

[juncai, 2016-05-26 18:43:52]

LGTM with nit.

https://codereview.chromium.org/2014933002/diff/20001/device/core/device_moni...
File device/core/device_monitor_linux.cc (right):

https://codereview.chromium.org/2014933002/diff/20001/device/core/device_moni...
device/core/device_monitor_linux.cc:56: monitor_fd_, true,
base::MessageLoopForIO::WATCH_READ,
nit: added comment to true parameter.

[Reilly Grant (use Gerrit), 2016-05-26 18:51:36]

Added comment for boolean parameter.

[Zachary Kuznia, 2016-05-26 21:08:42]

lgtm

[Reilly Grant (use Gerrit), 2016-05-26 22:29:02]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-05-26 22:29:02]

The patchset sent to the CQ was uploaded after l-g-t-m from juncai@chromium.org
Link to the patchset: https://codereview.chromium.org/2014933002/#ps40001
(title: "Added comment for boolean parameter.")

[commit-bot: I haz the power, 2016-05-26 22:29:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2014933002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2014933002/40001

[commit-bot: I haz the power, 2016-05-26 23:35:02]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-26 23:36:56]

Description was changed from

==========
Move DeviceMonitorLinux to //device/core.

This code will be shared by HidServiceLinux and UsbServiceLinux so it
needs to be in the shared //device/core component.

Sharing the service introduces an issue with destruction order that is
resolved by adding a WillDestroyMonitorMessageLoop() method to its
Observer interface.

BUG=422562
==========

to

==========
Move DeviceMonitorLinux to //device/core.

This code will be shared by HidServiceLinux and UsbServiceLinux so it
needs to be in the shared //device/core component.

Sharing the service introduces an issue with destruction order that is
resolved by adding a WillDestroyMonitorMessageLoop() method to its
Observer interface.

BUG=422562

Committed: https://crrev.com/57b6ff447084f1b0040557436d0f55c1e152626c
Cr-Commit-Position: refs/heads/master@{#396318}
==========

[commit-bot: I haz the power, 2016-05-26 23:36:57]

Patchset 3 (id:??) landed as
https://crrev.com/57b6ff447084f1b0040557436d0f55c1e152626c
Cr-Commit-Position: refs/heads/master@{#396318}