Web MIDI: dispatch MIDIConnectionEvent

Web MIDI: dispatch MIDIConnectionEvent

In the latest editor's draft spec, MIDIAccess has only one EventHandler
onstatechange that is integrated thing of onconnect and ondisconnect.
Also, ondisconnect of MIDIPort is also renamed to onstatechange.

See the latest editor]s draft spec:
  - http://webaudio.github.io/web-midi-api/#midiaccess-interface
  - http://webaudio.github.io/web-midi-api/#idl-def-MIDIPort

This patch update these EventHandler attributes, and also add
implementation to dispatch these events.

BUG=279097
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190653

[Takashi Toyoshima, 2015-02-21 17:12:19]

toyoshim@chromium.org changed reviewers:
+ kouhei@chromium.org

[Takashi Toyoshima, 2015-02-21 17:17:56]

Kouhei, I have some questions on this CL.
Please allow me to have some offline discussion.

My question is how can I obtain MIDIPort in MIDIConnectionEvent in the right
way.
I don't have enough confidence to own MIDIPort as
PersistentWillBeMember<MIDIPort> in MIDIConnectionEvent, and how to get a raw
pointer of MIDIPort from MIDIPortMap, or add a right creation interface to
MIDIConnectionEvent.

[kouhei (in TOK), 2015-02-23 01:40:41]

https://codereview.chromium.org/943173002/diff/1/Source/modules/webmidi/MIDIC...
File Source/modules/webmidi/MIDIConnectionEvent.h (right):

https://codereview.chromium.org/943173002/diff/1/Source/modules/webmidi/MIDIC...
Source/modules/webmidi/MIDIConnectionEvent.h:63: DEFINE_INLINE_VIRTUAL_TRACE()
DECLARE_VIRTUAL_TRACE is recommended over DEFINE_INLINE_VIRTUAL_TRACE where the
class has its corresponding cpp file.

[Takashi Toyoshima, 2015-02-23 11:15:35]

thanks. now it works fine.
please take another look.

https://codereview.chromium.org/943173002/diff/1/Source/modules/webmidi/MIDIC...
File Source/modules/webmidi/MIDIConnectionEvent.h (right):

https://codereview.chromium.org/943173002/diff/1/Source/modules/webmidi/MIDIC...
Source/modules/webmidi/MIDIConnectionEvent.h:63: DEFINE_INLINE_VIRTUAL_TRACE()
On 2015/02/23 01:40:41, kouhei wrote:
> DECLARE_VIRTUAL_TRACE is recommended over DEFINE_INLINE_VIRTUAL_TRACE where
the
> class has its corresponding cpp file.

Done.

[kouhei (in TOK), 2015-02-23 11:17:58]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2015-02-23 11:17:58]

lgtm

[commit-bot: I haz the power, 2015-02-23 11:18:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/943173002/20001

[commit-bot: I haz the power, 2015-02-23 13:08:53]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190653