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

Unified Diff: content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h

Issue 2217573002: bluetooth: Only add new devices, connected devices and devices that changed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@bluetooth-add-or-update
Patch Set: Fix typo Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h
diff --git a/content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h b/content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h
index 57ccefccee7e0d9da484c89e2cdbb1df6fb01d1a..1ab6e0cf601645a42fbd5ea55487707c7faff05c 100644
--- a/content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h
+++ b/content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h
@@ -143,6 +143,39 @@ class LayoutTestBluetoothAdapterProvider {
static scoped_refptr<testing::NiceMock<device::MockBluetoothAdapter>>
GetSecondDiscoveryFindsHeartRateAdapter();
+ // |DeviceEventAdapter|
+ // Inherits from |PoweredAdapter|
+ // Internal Structure:
+ // - Connected Heart Rate Device
+ // - IsGattConnected: Returns true.
+ // - UUIDs:
+ // - Heart Rate UUID (0x180d)
+ // - Changing Battery Device
+ // - IsGattConnected: Returns false.
+ // - No UUIDs (A Battery UUID (0x180f) is added by
+ // StartDiscoverySessionWithFilter).
+ // - Non Connected Tx Power Device
+ // - IsGattConnected: Returns false.
+ // - UUIDs:
+ // - Tx Power (0x1804)
+ // - Discovery Generic Access Device
+ // - IsGattConnected: Returns true.
+ // - No UUIDs (A Generic Access UUID (0x1800) is added by
+ // StartDiscoverySessionWithFilter).
+ // Mock Functions:
+ // - StartDiscoverySessionWithFilter: Performs the following steps the first
+ // time is called:
+ // 1. Post a task to add New Glucose Device (Contains a single
+ // Glucose UUID (0x1808) and no services).
+ // 2. Adds a Battery UUID to Changing Battery Device and posts a task
+ // that notifies observers that the device changed.
+ // 3. Adds a Generic Access UUID to Discovery Generic Access Device and
+ // posts a task to Notify its services have been discovered.
+ // 4. Return a discovery session.
+ // Successive calls just return a discovery session.
+ static scoped_refptr<testing::NiceMock<device::MockBluetoothAdapter>>
+ GetDeviceEventAdapter();
+
// |MissingServiceHeartRateAdapter|
// Inherits from |EmptyAdapter|
// Internal Structure:
@@ -421,7 +454,11 @@ class LayoutTestBluetoothAdapterProvider {
// - GetName:
// Returns: device_name.
// - IsPaired:
- // Returns true.
+ // Returns false.
+ // - IsConnected:
+ // Returns false.
+ // - IsGattConnected:
+ // Returns false.
// - ConnectGatt:
// Calls error callback with
// BluetoothDevice::ConnectErrorCode::ERROR_UNSUPPORTED_DEVICE.

Powered by Google App Engine
This is Rietveld 408576698