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

Unified Diff: content/public/browser/bluetooth_chooser.h

Issue 2245603003: Add signal strength indicator icon to WebBluetooth chooser on non-Mac desktops (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address more 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/public/browser/bluetooth_chooser.h
diff --git a/content/public/browser/bluetooth_chooser.h b/content/public/browser/bluetooth_chooser.h
index 84b5cbea65a806a3de3147a8138eaa052981ec2a..c250943fbe87edade405b5f9d254054862391090 100644
--- a/content/public/browser/bluetooth_chooser.h
+++ b/content/public/browser/bluetooth_chooser.h
@@ -74,14 +74,15 @@ class CONTENT_EXPORT BluetoothChooser {
// Sometimes when a Bluetooth device stops advertising, the |device_name| can
// be invalid, and in that case |should_update_name| will be set false.
//
- // Passing nullptr for |rssi| means that the device doesn't have RSSI which
- // happens when the device is already connected.
+ // The range of |signal_strength_level| is -1 to 4 inclusively.
+ // -1 means that the device doesn't have RSSI which happens when the device
+ // is already connected.
virtual void AddOrUpdateDevice(const std::string& device_id,
bool should_update_name,
const base::string16& device_name,
bool is_gatt_connected,
bool is_paired,
- const int8_t* rssi) {}
+ int signal_strength_level) {}
// Tells the chooser that a device is no longer available. The chooser should
// not call DeviceSelected() for a device that's been removed.
« no previous file with comments | « content/content_tests.gypi ('k') | content/shell/browser/layout_test/layout_test_bluetooth_chooser_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698