bluetooth: Return int8_t and use -128 for unknown tx power.

third_party/WebKit/Source/modules/bluetooth/BluetoothAdvertisingData.cpp

Index: third_party/WebKit/Source/modules/bluetooth/BluetoothAdvertisingData.cpp
diff --git a/third_party/WebKit/Source/modules/bluetooth/BluetoothAdvertisingData.cpp b/third_party/WebKit/Source/modules/bluetooth/BluetoothAdvertisingData.cpp
index 0fb0cb6ead6575eea71e4b27c5c30c3490c84ae2..a76841e8fef5ede6ee8b0a0e755c66de113fdbf5 100644
--- a/third_party/WebKit/Source/modules/bluetooth/BluetoothAdvertisingData.cpp
+++ b/third_party/WebKit/Source/modules/bluetooth/BluetoothAdvertisingData.cpp
@@ -7,9 +7,8 @@
 namespace blink {
 
 namespace {
-// TODO(ortuno): RSSI Unknown and Tx Power Unknown should have different
-// values. Add kUnknownTxPower when implemented: http://crbug.com/551572
-const int kUnknownPower = 127;
+    const int8_t kUnknownRSSI = 127;

[scheib, 2016/05/04 01:18:38]

Comment explaining the source of the numbers (cite spec as in
bluetooth_device.h)

[ortuno, 2016/05/04 16:44:12]

We removed BluetoothAdvertisingData so no need to repeat the value here.

+    const int8_t kUnknownTxPower = -128;
 } // namespace
 
 BluetoothAdvertisingData* BluetoothAdvertisingData::create(int8_t txPower, int8_t rssi)
@@ -19,7 +18,7 @@ BluetoothAdvertisingData* BluetoothAdvertisingData::create(int8_t txPower, int8_
 
 int8_t BluetoothAdvertisingData::txPower(bool& isNull)
 {
-    if (m_txPower == kUnknownPower) {
+    if (m_txPower == kUnknownTxPower) {
         isNull = true;
     }
     return m_txPower;
@@ -27,7 +26,7 @@ int8_t BluetoothAdvertisingData::txPower(bool& isNull)
 
 int8_t BluetoothAdvertisingData::rssi(bool& isNull)
 {
-    if (m_rssi == kUnknownPower) {
+    if (m_rssi == kUnknownRSSI) {
         isNull = true;
     }
     return m_rssi;