Change EventTarget callback APIs for add/RemoveEventListenerInternal.

third_party/WebKit/Source/modules/webmidi/MIDIInput.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 47 matching lines @@
 
 void MIDIInput::setOnmidimessage(EventListener* listener)
 {
     // Implicit open. It does nothing if the port is already opened.
     // See http://www.w3.org/TR/webmidi/#widl-MIDIPort-open-Promise-MIDIPort
     open();
 
     setAttributeEventListener(EventTypeNames::midimessage, listener);
 }
 
-bool MIDIInput::addEventListenerInternal(const AtomicString& eventType, EventListener* listener, const EventListenerOptions& options)
+void MIDIInput::addedEventListener(const AtomicString& eventType, const RegisteredEventListener& registeredListener)
 {
+    MIDIPort::addedEventListener(eventType, registeredListener);
     if (eventType == EventTypeNames::midimessage) {
         // Implicit open. See setOnmidimessage().
         open();

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

Ditto here, this used to be called on success or failure. Should have a local
expert look this over.

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

Added toyoshim@

     }
-    return EventTarget::addEventListenerInternal(eventType, listener, options);
 }
 
 void MIDIInput::didReceiveMIDIData(unsigned portIndex, const unsigned char* data, size_t length, double timeStamp)
 {
     DCHECK(isMainThread());
 
     if (!length)
         return;
 
     if (getConnection() != ConnectionStateOpen)
         return;
 
     // Drop sysex message here when the client does not request it. Note that this is not a security check but an
     // automatic filtering for clients that do not want sysex message. Also note that sysex message will never be sent
     // unless the current process has an explicit permission to handle sysex message.
     if (data[0] == 0xf0 && !midiAccess()->sysexEnabled())
         return;
     DOMUint8Array* array = DOMUint8Array::create(data, length);
     dispatchEvent(MIDIMessageEvent::create(timeStamp, array));
 }
 
 DEFINE_TRACE(MIDIInput)
 {
     MIDIPort::trace(visitor);
 }
 
 } // namespace blink