Web MIDI backend for Windows 10

media/midi/BUILD.gn

Index: media/midi/BUILD.gn
diff --git a/media/midi/BUILD.gn b/media/midi/BUILD.gn
index cfeb039968141daa27e546d6083a43844dc981fb..61d899f0db3b732200b1520e4cd2a6c17397f688 100644
--- a/media/midi/BUILD.gn
+++ b/media/midi/BUILD.gn
@@ -3,6 +3,7 @@
 # found in the LICENSE file.
 
 import("//build/config/android/config.gni")
+import("//build/config/win/midi_winrt.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//media/media_options.gni")
@@ -79,8 +80,6 @@ component("midi") {
     "midi_manager.h",
     "midi_manager_mac.cc",
     "midi_manager_mac.h",
-    "midi_manager_win.cc",
-    "midi_manager_win.h",
     "midi_message_queue.cc",
     "midi_message_queue.h",
     "midi_message_util.cc",
@@ -139,6 +138,14 @@ component("midi") {
 
   if (is_win) {
     deps += [ "//device/usb" ]

[Takashi Toyoshima, 2016/08/15 08:46:23]

Hum... this line 140 might be an existing bug. I may miss something, but current
Windows backend does not depend on //device/usb.

This is not your fault, but could you check if we can compile without this line?

[Shao-Chuan Lee, 2016/08/16 07:42:24]

Will have it removed in another CL.

+    sources += [ "midi_manager_win.h" ]

[Takashi Toyoshima, 2016/08/15 08:46:23]

I think we should have a dedicated header, _winrt.h, for for your new
implementation.

[Shao-Chuan Lee, 2016/08/16 07:42:24]

Done.

+
+    if (midi_winrt) {
+      sources += [ "midi_manager_winrt.cc" ]
+      libs += [ "runtimeobject.lib" ]

[Takashi Toyoshima, 2016/08/15 08:46:23]

Can you add TODO(crbug.com/512433) with comments for the line 145?
This would be a simplified version of your note in the CL description.

[Shao-Chuan Lee, 2016/08/16 07:42:24]

Done.

+    } else {
+      sources += [ "midi_manager_win.cc" ]
+    }
   }
 
   if (use_alsa && use_udev) {