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

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

Issue 2051333004: Implement BluetoothGattNotifySession::Stop on Android, 2nd attempt (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address new review comments 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.cc
diff --git a/content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.cc b/content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.cc
index 16f613fb9e262d26a27454eee54617e2a1454044..eb554d4d8595714f37a7b6c0ed4ead1fec41a9a3 100644
--- a/content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.cc
+++ b/content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.cc
@@ -1124,7 +1124,7 @@ LayoutTestBluetoothAdapterProvider::GetHeartRateService(
.WillByDefault(RunCallbackWithResult<0 /* success_callback */>(
[adapter, measurement_ptr]() {
std::unique_ptr<NiceMockBluetoothGattNotifySession> notify_session(
- GetBaseGATTNotifySession(measurement_ptr->GetIdentifier()));
+ GetBaseGATTNotifySession(measurement_ptr->GetWeakPtr()));
std::vector<uint8_t> rate(1 /* size */);
rate[0] = 60;
@@ -1253,9 +1253,9 @@ LayoutTestBluetoothAdapterProvider::GetErrorCharacteristic(
// static
std::unique_ptr<NiceMockBluetoothGattNotifySession>
LayoutTestBluetoothAdapterProvider::GetBaseGATTNotifySession(
- const std::string& characteristic_identifier) {
+ base::WeakPtr<device::BluetoothRemoteGattCharacteristic> characteristic) {
std::unique_ptr<NiceMockBluetoothGattNotifySession> session(
- new NiceMockBluetoothGattNotifySession(characteristic_identifier));
+ new NiceMockBluetoothGattNotifySession(characteristic));
ON_CALL(*session, Stop(_))
.WillByDefault(testing::DoAll(
« no previous file with comments | « content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h ('k') | device/bluetooth/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698