Change EventTarget callback APIs for add/RemoveEventListenerInternal.

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

Index: third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp
diff --git a/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp b/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp
index 921625d8c4f7ab15f1b643251ca5b28e966dfabf..7182d4177e94304e1b297a423049e7a4ec75ebb1 100644
--- a/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp
+++ b/third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTCharacteristic.cpp
@@ -91,15 +91,15 @@ ExecutionContext* BluetoothRemoteGATTCharacteristic::getExecutionContext() const
     return ActiveDOMObject::getExecutionContext();
 }
 
-bool BluetoothRemoteGATTCharacteristic::addEventListenerInternal(const AtomicString& eventType, EventListener* listener, const EventListenerOptions& options)
+void BluetoothRemoteGATTCharacteristic::addedEventListener(const AtomicString& eventType, const RegisteredEventListener& registeredListener)
 {
+    EventTargetWithInlineData::addedEventListener(eventType, registeredListener);
     // We will also need to unregister a characteristic once all the event
     // listeners have been removed. See http://crbug.com/541390
     if (eventType == EventTypeNames::characteristicvaluechanged) {
         WebBluetooth* webbluetooth = BluetoothSupplement::fromExecutionContext(getExecutionContext());

[bokan, 2016/05/03 15:52:46]

This used to get called even if the listener failed to be added. Seems to me
like that might have been an oversight but would you mind adding someone who
worked on this to double check?

[dtapuska, 2016/05/03 20:53:04]

Added ortuno@

[ortuno, 2016/05/03 20:59:56]

I wasn't aware addEventListenerInternal was called even if the registration
failed. So thanks for the fix!

         webbluetooth->registerCharacteristicObject(m_webCharacteristic->characteristicInstanceID, this);
     }
-    return EventTarget::addEventListenerInternal(eventType, listener, options);
 }
 
 class ReadValueCallback : public WebBluetoothReadValueCallbacks {