1201596 - Implement Windows Backend for WebMIDI

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, task, P3)

Description

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Implement platform specific MIDI backend for windows

Comment 1

[Adam Goode]

If Windows 10 is ok as a minimum version for this, then maybe Firefox can use the new MIDI API:
https://msdn.microsoft.com/library/windows/apps/windows.devices.midi.aspx

Comment 2

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Making this depend on bug 1135640 since we're doing midi platform specifics in Rust.

Comment 3

[Pomax]

It might make more sense to rely on rtmidi, https://github.com/thestk/rtmidi, which has been a proven solution for quite a while now. And as a cross-platform solution it would probably make it easier to get Web MIDI implemented for windows, osx, and linux all at the same time since it removed the need for platform-specific code in the FF codebase.

Comment 4

[Adam Goode]

The tricky part of Web MIDI is in the hotplugging and metadata handling of MIDI devices. rtmidi doesn't seem to implement hotplug.

My hope with the rust support library was that it would implement this platform specific stuff that no other library does.

Comment 5

[Pomax]

Hm, have you looked at http://www.tobias-erichsen.de/software/virtualmidi.html by any chance? It has an odd license to it (namely "you need my permission") but if Tobias allows its use that would probably solve the problem of dynamic devices on Windows all the way back to Windows XP x86

Comment 6

[Pomax]

Sorry, no, I think I misread you. That would not help with hotplugging physical devices, only with creating the virtual device side of things.

Comment 7

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

While it does sound a bit NIH, one of my goals with device interfaces in firefox is to keep everything at the systems level written in Rust, also. I have good reasons (based on bad experiences) for this. :)

Comment 8

[Paul Adenot (:padenot)]

Attachment: Bug 1201596 - Use the right struct for Gecko's timestamp on Windows. r?gsvelto