Bug 1188394 - Fix warning: NS_ENSURE_TRUE(aTypes.Length()) failed, r=joliu

dom/bluetooth/bluetooth2/BluetoothAdapter.cpp

@@ -1008,6 +1008,11 @@ BluetoothAdapter::HandlePropertyChanged(const BluetoothValue& aValue)
     }
   }
 
+  if (types.IsEmpty()) {
+    // No adapter attribute changed
+    return;
+  }
+
   DispatchAttributeEvent(types);
 }
 
@@ -1134,7 +1139,7 @@ BluetoothAdapter::HandleDeviceUnpaired(const BluetoothValue& aValue)
 void
 BluetoothAdapter::DispatchAttributeEvent(const Sequence<nsString>& aTypes)
 {
-  NS_ENSURE_TRUE_VOID(aTypes.Length());
+  MOZ_ASSERT(!aTypes.IsEmpty());
 
   BluetoothAttributeEventInit init;
   init.mAttrs = aTypes;

dom/bluetooth/bluetooth2/BluetoothAdapter.h

@@ -285,6 +285,8 @@ private:
 
   /**
    * Fire BluetoothAttributeEvent to trigger onattributechanged event handler.
+   *
+   * @param aTypes [in] Array of changed attributes. Must be non-empty.
    */
   void DispatchAttributeEvent(const Sequence<nsString>& aTypes);
 

dom/bluetooth/bluetooth2/BluetoothDevice.cpp

@@ -293,13 +293,18 @@ BluetoothDevice::HandlePropertyChanged(const BluetoothValue& aValue)
     }
   }
 
+  if (types.IsEmpty()) {
+    // No device attribute changed
+    return;
+  }
+
   DispatchAttributeEvent(types);
 }
 
 void
 BluetoothDevice::DispatchAttributeEvent(const Sequence<nsString>& aTypes)
 {
-  NS_ENSURE_TRUE_VOID(aTypes.Length());
+  MOZ_ASSERT(!aTypes.IsEmpty());
 
   BluetoothAttributeEventInit init;
   init.mAttrs = aTypes;

dom/bluetooth/bluetooth2/BluetoothDevice.h

@@ -117,6 +117,8 @@ private:
 
   /**
    * Fire BluetoothAttributeEvent to trigger onattributechanged event handler.
+   *
+   * @param aTypes [in] Array of changed attributes. Must be non-empty.
    */
   void DispatchAttributeEvent(const Sequence<nsString>& aTypes);