[WebMIDI] Introduce MidiManagerUsb

media/midi/midi_manager_usb.h

Index: media/midi/midi_manager_usb.h
diff --git a/media/midi/midi_manager_usb.h b/media/midi/midi_manager_usb.h
new file mode 100644
index 0000000000000000000000000000000000000000..2b845b0b2932f13ef8bda0d6b9043ad3be0f5234
--- /dev/null
+++ b/media/midi/midi_manager_usb.h
@@ -0,0 +1,83 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef MEDIA_MIDI_MIDI_MANAGER_USB_H_
+#define MEDIA_MIDI_MIDI_MANAGER_USB_H_
+
+#include <utility>
+#include <vector>
+
+#include "base/basictypes.h"
+#include "base/bind.h"
+#include "base/callback.h"
+#include "base/containers/hash_tables.h"
+#include "base/memory/scoped_ptr.h"
+#include "media/midi/midi_manager.h"
+#include "media/midi/usb_midi_device.h"
+#include "media/midi/usb_midi_input_stream.h"
+#include "media/midi/usb_midi_jack.h"
+#include "media/midi/usb_midi_output_stream.h"
+
+namespace media {
+
+// MIDIManager for USB-MIDI.
+class MIDIManagerUsb : public MIDIManager,

[scherkus (not reviewing), 2014/01/27 19:23:49]

nit: I'd like to see you folks agree on a naming convention

for example ... you have MIDI vs. Midi, not to mention Usb ... but no USB yet ;)

looking at the rest of the code it looks like you prefer Midi

[yhirano, 2014/01/28 04:31:05]

Discussed with toyoshim@.
We are going to change MIDI to Midi in future CLs.
In this CL, I renamed things I introduced such as MidiManagerUsb.

+                       public UsbMidiDeviceDelegate,
+                       public UsbMidiInputStream::Delegate {
+ public:
+  explicit MIDIManagerUsb(scoped_ptr<UsbMidiDevice::Factory> device_factory);
+  virtual ~MIDIManagerUsb();
+
+  // MIDIManager implementation.
+  virtual bool Initialize() OVERRIDE;
+  virtual void DispatchSendMIDIData(MIDIManagerClient* client,
+                                    uint32 port_index,
+                                    const std::vector<uint8>& data,
+                                    double timestamp) OVERRIDE;
+
+  // UsbMidiDeviceDelegate implementation.
+  virtual void ReceiveUsbMidiData(UsbMidiDevice* device,
+                                  int endpoint_number,
+                                  const uint8* data,
+                                  size_t size,
+                                  double timestamp) OVERRIDE;
+
+  // UsbMidiInputStream::Delegate implementation.
+  virtual void OnReceivedData(size_t jack_index,
+                              const uint8* data,
+                              size_t size,
+                              double timestamp) OVERRIDE;
+
+  const ScopedVector<UsbMidiOutputStream>& output_streams() const {
+    return output_streams_;
+  }
+  const UsbMidiInputStream* input_stream() const { return input_stream_.get(); }
+
+  // Initializes this object.
+  // When the initialization finishes, |callback| will be called with the
+  // result.
+  // When this factory is destroyed during the operation, the operation
+  // will be canceled silently (i.e. |callback| will not be called).
+  void Initialize(base::Callback<void(bool result)> callback);
+
+ private:
+  static void OnRequestPermissionDone(bool result);

[scherkus (not reviewing), 2014/01/27 19:23:49]

I can't find this function -- remove?

[yhirano, 2014/01/28 04:31:05]

Done.

+  void OnEnumerateDevicesDone(bool result, UsbMidiDevice::Devices* devices);
+
+  scoped_ptr<UsbMidiDevice::Factory> device_factory_;
+  ScopedVector<UsbMidiDevice> devices_;
+  ScopedVector<UsbMidiOutputStream> output_streams_;
+  scoped_ptr<UsbMidiInputStream> input_stream_;
+
+  base::Callback<void(bool result)> initialize_callback_;
+
+  // A map from <endpoint_number, cable_number> to the index of input jacks.
+  base::hash_map<std::pair<int, int>, size_t> input_jack_dictionary_;
+
+  DISALLOW_COPY_AND_ASSIGN(MIDIManagerUsb);
+};
+
+}  // namespace media
+
+#endif  // MEDIA_MIDI_MIDI_MANAGER_USB_H_