Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1039)

Unified Diff: media/midi/midi_manager_alsa.cc

Issue 662233002: Web MIDI: make MidiManagerMac notify device connections (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase + cleanup Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/midi/midi_manager_alsa.cc
diff --git a/media/midi/midi_manager_alsa.cc b/media/midi/midi_manager_alsa.cc
index 492573b13de4c42fe3d679dc3bad521d1b42af5d..17b32f5b2f1fd6bf9b48f48253168a579cdad85f 100644
--- a/media/midi/midi_manager_alsa.cc
+++ b/media/midi/midi_manager_alsa.cc
@@ -252,7 +252,9 @@ void MidiManagerAlsa::StartInitialization() {
VLOG(1) << "snd_seq_subscribe_port fails: " << snd_strerror(err);
} else {
source_map_[AddrToInt(sender)] = current_input++;
- AddInputPort(MidiPortInfo(id, manufacturer, name, version));
+ const bool connected = trie;
yhirano 2014/10/24 03:30:08 true
Takashi Toyoshima 2015/02/17 17:10:37 Done.
+ AddInputPort(
+ MidiPortInfo(id, manufacturer, name, version, connected));
}
}
if ((caps & kRequiredOutputPortCaps) == kRequiredOutputPortCaps) {
@@ -286,7 +288,9 @@ void MidiManagerAlsa::StartInitialization() {
snd_midi_event_new(kSendBufferSize, &encoder);
encoders_.push_back(encoder);
out_ports_.push_back(out_port);
- AddOutputPort(MidiPortInfo(id, manufacturer, name, version));
+ const bool connected = true;
+ AddOutputPort(
+ MidiPortInfo(id, manufacturer, name, version, connected));
}
}
}

Powered by Google App Engine
This is Rietveld 408576698