MIDIMessageEvent data is not nullable

[padenot]

STR in Chrome:

> var event = new MIDIMessageEvent("something");
> console.log(event.data);
< null

MIDIMessageEvent.data is not nullable, so this should be an empty array. It's harmless to allow an empty MIDIMessageEvent but it's not particularly useful either. We can change the spec for compat instead of having Chrome fixed maybe.

[hoch]

partial interface MIDIMessageEvent : Event {
  readonly attribute Uint8Array? data;
}

Does this look good? I agree that null is better.

[padenot]

dictionary MIDIMessageEventInit: EventInit {
  Uint8Array? data;
};

? is implied, the opposite being to specify require iirc.

[hoch]

Yes. Then we can lose the dictionary bit. I edited my comment above. Thanks!

[padenot]

https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/modules/webmidi/midi_message_event_init.idl;l=8-11 a related TODO in Chromium.

[padenot]

#168 seems to be a dupe, I'll sort it out.

[mjwilson-google]

Fixed by #252