| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/944153002
Cr-Commit-Position: refs/heads/master@{#317455}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix errors. Most of the problems were in net. Moves CreateAndConnectStreamForTesting's declaration to the header corresponding to the file it's implemented in.
Adds the other directories that don't pass yet to the list commented out so
it's easier to see what's still left.
TBR=armansito@chromium.org
Review URL: https://codereview.chromium.org/936953004
Cr-Commit-Position: refs/heads/master@{#317425}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Windows HID library requires output report buffers to be exactly
the maximum length specified in the OutputReportByteLength field of
HIDP_CAPS. This patch automatically pads output reports to this length
with zeros and rejects reports longer than this length.
BUG=439775
Review URL: https://codereview.chromium.org/943783003
Cr-Commit-Position: refs/heads/master@{#317393}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch implements the estimation of (dis)chargingTime on devices
running Android L (and above). Android L supports new properties in
BatteryManager which can be used to provide such estimates.
BUG=401553
TEST=http://jsbin.com/battery-status-test
Review URL: https://codereview.chromium.org/874713003
Cr-Commit-Position: refs/heads/master@{#317372}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
methods.
Initial patch in https://codereview.chromium.org/848613003 used
DCHECK in too many methods that should function even when
the adapter is not present.
BUG=452705
Review URL: https://codereview.chromium.org/933823003
Cr-Commit-Position: refs/heads/master@{#317332}
|
|
|
|
|
|
|
|
|
|
|
| |
BatteryStatusObserver::Stop can be called twice from
BatteryStatusService::Shutdown, so power_handle_ and batter_change_handle_
should be set to NULL on the first call, otherwise the second call
will pass invalid handles to UnregisterNotification.
Review URL: https://codereview.chromium.org/924193006
Cr-Commit-Position: refs/heads/master@{#317321}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/940613002/)
Reason for revert:
This broken win8 gn bots because of an illegal include.
ERROR at //device/usb/usb_service_impl.cc:21:11: Include not allowed.
#include "device/core/device_monitor_win.h"
e.g.
https://build.chromium.org/p/chromium.win/builders/Win8%20GN/builds/4896/steps/gn%20check/logs/stdio
Original issue's description:
> Add more targets to "gn check"
>
> R=dpranke
> TBR=armansito@chromium.org (bluetooth)
>
> Committed: https://crrev.com/8cac5f9d9c3847657b57783d30f861adb6c29cac
> Cr-Commit-Position: refs/heads/master@{#317160}
TBR=dpranke@chromium.org,armansito@chromium.org,brettw@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/941893002
Cr-Commit-Position: refs/heads/master@{#317204}
|
|
|
|
|
|
|
|
|
| |
R=dpranke
TBR=armansito@chromium.org (bluetooth)
Review URL: https://codereview.chromium.org/940613002
Cr-Commit-Position: refs/heads/master@{#317160}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Avoid possibly long blocking times in the Windows SetupDi function
calls made to enumerate devices by offloading to the FILE thread.
Now that we can be sure that a stray request to the device won't
block the UI additional calls to get the device's product and serial
number strings can be made.
BUG=457899
Review URL: https://codereview.chromium.org/936603003
Cr-Commit-Position: refs/heads/master@{#316939}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
and media-related overrides.
This CL implements delegate overrides for
BluetoothMediaEndpointServiceProvider and observer overrides for
BluetoothMediaClient and BluetoothMediaTransportClient. These
overrides will be called once a remote device connects to a local
A2DP audio sink, and the state of the audio sink should become idle.
The behaviors of FakeBluetoothMediaTransportClient,
FakeBluetoothMediaEndpointServiceProvider and
FakeMediaEndpointServiceProvider are also defined for
connection-related tests in this CL.
BUG=441581
TEST=device_unittests --gtest_filter=*AudioSink*
Review URL: https://codereview.chromium.org/910023002
Cr-Commit-Position: refs/heads/master@{#316709}
|
|
|
|
|
|
|
|
| |
Obj-C 1.0 only allows to declare ivars in @interface section.
Review URL: https://codereview.chromium.org/917063004
Cr-Commit-Position: refs/heads/master@{#316236}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#13 id:230001 of https://codereview.chromium.org/912083003/)
Reason for revert:
Mac GN build fails on buildbot:
http://build.chromium.org/p/chromium.mac/builders/Mac%20GN%20%28dbg%29/builds/4172/steps/compile/logs/stdio
Original issue's description:
> device: Add GN version of the device_unittests target.
>
> Tested with the following command lines:
>
> $ gn gen out_gn/Debug
> $ ninja -C out_gn/Debug device_unittests
> $ out_gn/Debug/device_unittests
>
> BUG=None
> TEST=see above
> R=rockot@chromium.org
>
> Committed: https://crrev.com/e16ab4ff264bc3f6c2a850d820b5f5c79fe59d05
> Cr-Commit-Position: refs/heads/master@{#316058}
TBR=rockot@chromium.org,jam@chromium.org,tfarina@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None
Review URL: https://codereview.chromium.org/924613004
Cr-Commit-Position: refs/heads/master@{#316072}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Tested with the following command lines:
$ gn gen out_gn/Debug
$ ninja -C out_gn/Debug device_unittests
$ out_gn/Debug/device_unittests
BUG=None
TEST=see above
R=rockot@chromium.org
Review URL: https://codereview.chromium.org/912083003
Cr-Commit-Position: refs/heads/master@{#316058}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/911343002
Cr-Commit-Position: refs/heads/master@{#315911}
|
|
|
|
|
|
|
|
|
|
|
| |
This allows persistent serial connections to be supported on the
mojo-service-backed serial implementation.
BUG=389016
Review URL: https://codereview.chromium.org/908153002
Cr-Commit-Position: refs/heads/master@{#315514}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When failing to connect to a previously-paired device, the profile would
not be cleared. This sets adapter_ in RegisterProfile() so that it will
be set correctly for profile release later.
Also includes unit tests, adding a paired but non-connectable device.
BUG=455868
Review URL: https://codereview.chromium.org/910433002
Cr-Commit-Position: refs/heads/master@{#315394}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
of https://codereview.chromium.org/791763005/)
Reason for revert:
Breaks compile:
http://build.chromium.org/p/chromium.mac/builders/Mac%20Builder/builds/17750
../../device/bluetooth/bluetooth_low_energy_discovery_manager_mac.mm:119:30: error: class method '+UUIDWithString:' not found (return type defaults to 'id') [-Werror,-Wobjc-method-access]
CBUUID* uuid = [CBUUID UUIDWithString:uuidString];
^~~~~~~~~~~~~~
../../device/bluetooth/bluetooth_low_energy_device_mac.mm:82:39: error: use of undeclared identifier 'CBAdvertisementDataIsConnectable'
[advertisementData objectForKey:CBAdvertisementDataIsConnectable];
^
../../device/bluetooth/bluetooth_low_energy_device_mac.mm:85:39: error: use of undeclared identifier 'CBAdvertisementDataServiceDataKey'
[advertisementData objectForKey:CBAdvertisementDataServiceDataKey];
^
2 errors generated.
Original issue's description:
> Added bluetooth LE support on Mac platform
>
> BUG=449682
>
> Committed: https://crrev.com/42f79f8b08a99190627f3a67e86a378987104a4a
> Cr-Commit-Position: refs/heads/master@{#315190}
TBR=armansito@chromium.org,avi@chromium.org,groby@chromium.org,dvh@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=449682
Review URL: https://codereview.chromium.org/871843009
Cr-Commit-Position: refs/heads/master@{#315191}
|
|
|
|
|
|
|
|
| |
BUG=449682
Review URL: https://codereview.chromium.org/791763005
Cr-Commit-Position: refs/heads/master@{#315190}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL ports the GATT API bindings from the D-Bus API in our Chrome OS fork to
the API definition in doc/gatt-api.txt in BlueZ upstream.
TBR=keybuk@chromium.org
BUG=440968
Review URL: https://codereview.chromium.org/788193004
Cr-Commit-Position: refs/heads/master@{#315095}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.
This patch was automatically generated with an OS=android build using a
variation of https://codereview.chromium.org/598073004.
BUG=417463
R=timvolodine@chromium.org
Review URL: https://codereview.chromium.org/902973002
Cr-Commit-Position: refs/heads/master@{#315079}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds an additional function to the UsbService::Observer
interface that should be used to perform cleanup operations when a USB
device is disconnected. This is separate from the OnDeviceRemoved
function so that implementations of that function, which may depend on
other subsystems' knowledge of a connected device, can run before any
cleanup actions have taken place.
A browsertest for the chrome.usb.getUserSelectedDevices is included
which demonstrates the need for this 2-phase event handling.
BUG=452298
Review URL: https://codereview.chromium.org/891853002
Cr-Commit-Position: refs/heads/master@{#314973}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mechanical change that adds instrumentation required to locate the source of
jankiness (i.e. a long-running fragment of code executed as a part of the task
that causes jank) in the code. See the bug for details on what kind of jank we
are after. A number of similar CLs were landed, and none of them caused issues.
The code of the instrumentation is highly optimized and is not expected to
affect performance. The code simply creates a diagnostic task which is identical
to ones created by PostTask or IPC message handlers.
This also updates the existing instrumentation for bug 418183 to include the bug
number everywhere.
BUG=418183,455423,455434,455469
TEST=none
R=vadimt@chromium.org
TBR=jochen, mmenke, rockot, thestig
Review URL: https://codereview.chromium.org/901773002
Cr-Commit-Position: refs/heads/master@{#314885}
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=451321
Committed: https://crrev.com/bc78a2320b50eba4fe397f1f838a82ad30b16cd6
Cr-Commit-Position: refs/heads/master@{#314231}
Review URL: https://codereview.chromium.org/889283002
Cr-Commit-Position: refs/heads/master@{#314773}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:20001 of https://codereview.chromium.org/889283002/)
Reason for revert:
Switching to two pipes broke FIFO assumption, causing DataSinkTest.CancelDuringError and SerialConnectionTest.Cancel to become flaky
Original issue's description:
> Remove Client= from device/serial/data_stream.mojom.
>
> BUG=451321
>
> Committed: https://crrev.com/bc78a2320b50eba4fe397f1f838a82ad30b16cd6
> Cr-Commit-Position: refs/heads/master@{#314231}
TBR=rockot@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=451321
Review URL: https://codereview.chromium.org/873293006
Cr-Commit-Position: refs/heads/master@{#314524}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL implements Register function for BluetoothAudioSinkChromeOS
and comes up with the corresponding tests. For the tests,
FakeBluetoothMediaClient is changed to simulate
the behavior of the
real D-Bus APIs.
BUG=441581
TEST=device_unittests --gtest_filter=*AudioSink*
Review URL: https://codereview.chromium.org/876153002
Cr-Commit-Position: refs/heads/master@{#314244}
|
|
|
|
|
|
|
|
| |
BUG=451321
Review URL: https://codereview.chromium.org/889283002
Cr-Commit-Position: refs/heads/master@{#314231}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Profiles in bluez are not specific to a device.
When client code tries to connect to two different devices, only
one profile is created, managed by BluetoothAdapterChromeOS.
dbus messages are multiplexed based on the device desired using new
class BluetoothAdapterProfileChromeOS.
This is a resubmit with bugfixes after https://codereview.chromium.org/851123002/ was reverted by
https://codereview.chromium.org/868753006/
BUG=421207
Review URL: https://codereview.chromium.org/888913002
Cr-Commit-Position: refs/heads/master@{#314030}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BluetoothDispatcherHost is extended to supply all the
attributes of the BluetoothDevice web API.
Also, add device::BluetoothDevice::VENDOR_ID_MAX_VALUE for
use in the IPC message.
Depends on https://codereview.chromium.org/882223002/ to
define new attributes on WebBluetoothDevice, and will not
be visible to script until
https://codereview.chromium.org/876623003/
BUG=438305
Review URL: https://codereview.chromium.org/885723002
Cr-Commit-Position: refs/heads/master@{#313999}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:80001 of https://codereview.chromium.org/851123002/)
Reason for revert:
This causes a leak on Chrome Os ASan.
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/6306/steps/device_unittests/logs/Connect
Original issue's description:
> Manage profiles in BluetoothAdapter on ChromeOS
>
> Profiles in bluez are not specific to a device.
> When client code tries to connect to two different devices, only
> one profile is created, managed by BluetoothAdapterChromeOS.
>
> dbus messages are multiplexed based on the device desired using new
> class BluetoothAdapterProfileChromeOS.
>
> BUG=chromium:421207
>
> Committed: https://crrev.com/c56186ac2901c99349cbb4b4a6b8aa0ff8088c4d
> Cr-Commit-Position: refs/heads/master@{#313837}
TBR=armansito@chromium.org,keybuk@chromium.org,rpaquay@chromium.org,jamuraa@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:421207
Review URL: https://codereview.chromium.org/868753006
Cr-Commit-Position: refs/heads/master@{#313883}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Profiles in bluez are not specific to a device.
When client code tries to connect to two different devices, only
one profile is created, managed by BluetoothAdapterChromeOS.
dbus messages are multiplexed based on the device desired using new
class BluetoothAdapterProfileChromeOS.
BUG=chromium:421207
Review URL: https://codereview.chromium.org/851123002
Cr-Commit-Position: refs/heads/master@{#313837}
|
|
|
|
|
|
|
|
|
|
|
|
| |
armansito@ is already an owner of device/bluetooth and device/nfc, so
make him and OWNER for the occasional unit test additions to device_tests.gyp.
BUG=none
R=rpaquay@chromium.org
Review URL: https://codereview.chromium.org/886443003
Cr-Commit-Position: refs/heads/master@{#313823}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change merges the initial call to SetCommState/tcsetattr with
the call to ConfigurePort so that parameters such as baud rate are
configured immediately when the port is opened. Default values for
these parameters are now always applied.
This works around an issue with some serial adapter drivers on Windows
that report an invalid baud rate through GetCommState. Having the
intended baud rate available for the first call to SetCommState allows
an application to provide a valid value so that Chrome does not have
to guess.
BUG=448407
Review URL: https://codereview.chromium.org/873903002
Cr-Commit-Position: refs/heads/master@{#313604}
|
|
|
|
|
|
|
|
|
|
|
|
| |
C++11 range-based for loops make it a lot easier to iterate over an STL
container without incredibly long type names. The last users of these
typedefs have been updated and so they are no longer necessary.
BUG=
Review URL: https://codereview.chromium.org/869063004
Cr-Commit-Position: refs/heads/master@{#312949}
|
|
|
|
|
|
|
|
|
|
|
|
| |
As a followup to the move of the Mojo SDK and EDK to live under //third_party,
this CL updates Chromium code to include SDK/EDK headers via fully-qualified
paths instead of relying on include_dirs that are exported by the Mojo targets.
This CL also includes mechanical updates to DEPS files to satisfy checkdeps.
Review URL: https://codereview.chromium.org/846443006
Cr-Commit-Position: refs/heads/master@{#312858}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/837303003
Cr-Commit-Position: refs/heads/master@{#312811}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
First, change 13cf2cfcb5325ecfff98ee8ccde723f22e606d22 introduced a
logic error into AndroidUsbDevice::EnumerateOnFileThread that caused
enumerations to never complete because the BarrierClosure was not
called the correct number of times.
Second, this change may have also exposed an issue (investigated
before) where a libusb_transfer can be cancelled twice, once in
ReleaseInterface and again in CloseInternal. The second cancellation
occurs after the interface has been released and so an invalid handle
is passed to AbortPipe. A cancelled_ member has been added to Transfer
to prevent this from happening.
Some conversions to C++11 range loops (added during debugging to make
the loops clearer) are preserved in this change.
BUG=450007
Review URL: https://codereview.chromium.org/864183002
Cr-Commit-Position: refs/heads/master@{#312796}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL implements Bluetooth Audio Sink API for ChromeOS platform,
where user applications can register/unregister a BT audio sink
via BlueAdapter interface. This API acts as a middle layer hiding
the components of BlueZ/D-Bus.
BUG=441581
Review URL: https://codereview.chromium.org/787743002
Cr-Commit-Position: refs/heads/master@{#312562}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These tests are essentially copies of the HID device add and remove
event tests. In order to reconcile issues with the MockUsbService
lifetime it is now tracked by a MessageLoop::DestructionObserver at the
generic UsbService level. This is similar to the pattern used by the
HidService until it was moved to the UI thread and could use a
LazyInstance.
BUG=411715
Review URL: https://codereview.chromium.org/800963005
Cr-Commit-Position: refs/heads/master@{#312538}
|
|
|
|
|
|
|
|
|
|
|
| |
Renamed the enum members declared as ERRORCODE_* to ERROR_* to be consistent
with the rest of the code.
BUG=none
Review URL: https://codereview.chromium.org/839573003
Cr-Commit-Position: refs/heads/master@{#312446}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This tells the linker to resolve symbols for shared libraries at build
time, not run time. This alerts developers much earlier that their
dependencies are underspecified.
BUG=371125
Review URL: https://codereview.chromium.org/843583006
Cr-Commit-Position: refs/heads/master@{#312317}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add missing dependencies that caused build failures with this flag.
(I did similar changes in the BUILD.gn files too, but blindly.)
BUG=371125
R=derat@chromium.org, thestig@chromium.org
TBR=antrim, bshe, thestig, timvolodine
Review URL: https://codereview.chromium.org/857353004
Cr-Commit-Position: refs/heads/master@{#312280}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously only simple mock responses were supported in
BluetoothDispatcherHost. This change adds use of an actual adapter,
and enables the first actual bluetooth.requestDevices responses
to function in the browser (with the experimental-web-platform-features
flag).
BUG=420284
Review URL: https://codereview.chromium.org/787953004
Cr-Commit-Position: refs/heads/master@{#312241}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL move //mojo/public and //mojo/edk to live in the following locations:
- //third_party/mojo/src/mojo/public
- //third_party/mojo/src/mojo/edk
It moves the related gypfiles from //mojo to //third_party/mojo and updates
them as necessary to account for the file moves. It also updates clients of the
mojo SDK and EDK targets in both GYP and GN. (Note that for GN, the mojo SDK
and EDK build systems are maintained in the Mojo repo and designed to be
flexible wrt the location of the SDK/EDK in a client repo, so no changes are
needed.
This CL does not update include paths to the code being moved to limit the
number of moving parts, instead relying on the include_dirs that the SDK and
EDK targets supply to their direct dependents to ensure that include paths
continue to resolve correctly.
NOPRESUBMIT=true
Review URL: https://codereview.chromium.org/814543006
Cr-Commit-Position: refs/heads/master@{#312129}
|
|
|
|
|
|
|
|
|
|
| |
from the generic profile of a bluetooth low energy device.
BUG=449678
Review URL: https://codereview.chromium.org/851323004
Cr-Commit-Position: refs/heads/master@{#312000}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The BluetoothAdapter is a lazy instance that may live due to references
until the shutdown of the browser. On ChromeOS the adapter and related
classes must release references to the DBusThreadManager before it is
shutdown. This patch adds an explicit call for that shutdown.
BUG=447387
Review URL: https://codereview.chromium.org/848613003
Cr-Commit-Position: refs/heads/master@{#311986}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.
This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.
BUG=417463
R=reillyg@chromium.org
Review URL: https://codereview.chromium.org/799933008
Cr-Commit-Position: refs/heads/master@{#311826}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL adds RegisterAudioSink method to Bluetooth adapter
abstraction class and its subclasses.For ChromeOS, the
implementation will be in the following CL. As for Mac and Windows,
there is no implementation for now. This CL also adds
BluetoothAudioSink::ErrorCode and BluetoothAudioSink::Unregister.
BUG=441581
Review URL: https://codereview.chromium.org/782293004
Cr-Commit-Position: refs/heads/master@{#311724}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change includes the raw HID report descriptor in the HidDeviceInfo
class and exposes it to apps and extensions using the chrome.hid API.
For simplicity on OS X the platform APIs for collecting information from
the report descriptor are no longer used and instead the parsing code
from Linux is used. On Windows the raw HID report descriptor is not
available because Windows exposes HID devices on a per-top-level
collection basis.
BUG=442818
Review URL: https://codereview.chromium.org/801833003
Cr-Commit-Position: refs/heads/master@{#311192}
|
|
|
|
|
|
|
|
|
|
|
| |
Silence -Wdelete-non-virtual-dtor warnings on this class by marking it
final and inheriting from DeviceMonitorWin::Observe privately.
BUG=
Review URL: https://codereview.chromium.org/850563002
Cr-Commit-Position: refs/heads/master@{#311188}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change documents that these helper classes live on the FILE thread
by naming them HidServiceLinux::FileThreadHelper and
HidConnectionLinux::FileThreadHelper.
BUG=
Review URL: https://codereview.chromium.org/851543002
Cr-Commit-Position: refs/heads/master@{#311120}
|