Web MIDI: add blink APIs to notify device connection events

Source/modules/webmidi/MIDIAccessInitializer.cpp

Index: Source/modules/webmidi/MIDIAccessInitializer.cpp
diff --git a/Source/modules/webmidi/MIDIAccessInitializer.cpp b/Source/modules/webmidi/MIDIAccessInitializer.cpp
index b1d1013dac8e0aed3bef0d3238bbb74611a09c3a..60a53e25cd16e416506ad53e10a5672fdbb8ad11 100644
--- a/Source/modules/webmidi/MIDIAccessInitializer.cpp
+++ b/Source/modules/webmidi/MIDIAccessInitializer.cpp
@@ -55,16 +55,26 @@ ScriptPromise MIDIAccessInitializer::start()
     return promise;
 }
 
-void MIDIAccessInitializer::didAddInputPort(const String& id, const String& manufacturer, const String& name, const String& version)
+void MIDIAccessInitializer::didAddInputPort(const String& id, const String& manufacturer, const String& name, const String& version, bool active)
 {
     ASSERT(m_accessor);
-    m_portDescriptors.append(PortDescriptor(id, manufacturer, name, MIDIPort::MIDIPortTypeInput, version));
+    m_portDescriptors.append(PortDescriptor(id, manufacturer, name, MIDIPort::MIDIPortTypeInput, version, active));
 }
 
-void MIDIAccessInitializer::didAddOutputPort(const String& id, const String& manufacturer, const String& name, const String& version)
+void MIDIAccessInitializer::didAddOutputPort(const String& id, const String& manufacturer, const String& name, const String& version, bool active)
 {
     ASSERT(m_accessor);
-    m_portDescriptors.append(PortDescriptor(id, manufacturer, name, MIDIPort::MIDIPortTypeOutput, version));
+    m_portDescriptors.append(PortDescriptor(id, manufacturer, name, MIDIPort::MIDIPortTypeOutput, version, active));
+}
+
+void MIDIAccessInitializer::didSetInputPortState(unsigned portIndex, bool active)
+{
+    ASSERT_NOT_REACHED();

[yhirano, 2014/10/20 02:14:10]

Is this correct? IIUC the browser can fire this notification anytime, right?

[Takashi Toyoshima, 2014/10/20 02:40:25]

Embedder (Chromium) sends, didAddInputPort(), didAddOutputPort(), and
didStartSession() in this order. After that, didSetOutputPort() and
didSetInputPort() are used when a device online state is changed.

IIRC, once didStartSession() is called, these client methods are handled by
another class directly, right?

[Takashi Toyoshima, 2014/10/20 05:39:50]

I added comments here too.

+}
+
+void MIDIAccessInitializer::didSetOutputPortState(unsigned portIndex, bool active)
+{
+    ASSERT_NOT_REACHED();

[yhirano, 2014/10/20 02:14:10]

ditto

[Takashi Toyoshima, 2014/10/20 05:39:50]

ditto

 }
 
 void MIDIAccessInitializer::didStartSession(bool success, const String& error, const String& message)