|
Web MIDI: move MIDI message validating logic into message_util.{cc|h}
There are two motivations for this changes:
1. To reduce duplicated message handling code
2. To simplify MidiHost to make coming mojofication change as much
simple as possible
Also this change rename midi_message_util to message_util
since now that midi has a dedicated namespace.
BUG= 339749, 582327
Committed: https://crrev.com/698d79dd86089e67c1f678147f3172b4add13312
Cr-Commit-Position: refs/heads/master@{#426454}
Total comments: 6
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+287 lines, -301 lines) |
Patch |
|
M |
content/browser/media/midi_host.h
|
View
|
|
1 chunk |
+0 lines, -8 lines |
0 comments
|
Download
|
|
M |
content/browser/media/midi_host.cc
|
View
|
|
5 chunks |
+7 lines, -50 lines |
0 comments
|
Download
|
|
M |
content/browser/media/midi_host_unittest.cc
|
View
|
|
2 chunks |
+0 lines, -76 lines |
0 comments
|
Download
|
|
M |
media/midi/BUILD.gn
|
View
|
|
3 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
|
A |
media/midi/message_util.h
|
View
|
|
1 chunk |
+52 lines, -0 lines |
0 comments
|
Download
|
|
A |
media/midi/message_util.cc
|
View
|
1
2
|
1 chunk |
+103 lines, -0 lines |
0 comments
|
Download
|
|
A |
media/midi/message_util_unittest.cc
|
View
|
|
1 chunk |
+108 lines, -0 lines |
0 comments
|
Download
|
|
M |
media/midi/midi_manager_win.cc
|
View
|
1
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
media/midi/midi_message_queue.cc
|
View
|
1
|
4 chunks |
+11 lines, -28 lines |
0 comments
|
Download
|
|
D |
media/midi/midi_message_util.h
|
View
|
|
1 chunk |
+0 lines, -36 lines |
0 comments
|
Download
|
|
D |
media/midi/midi_message_util.cc
|
View
|
|
1 chunk |
+0 lines, -51 lines |
0 comments
|
Download
|
|
D |
media/midi/midi_message_util_unittest.cc
|
View
|
|
1 chunk |
+0 lines, -46 lines |
0 comments
|
Download
|
|
M |
media/midi/usb_midi_output_stream.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
Total messages: 22 (12 generated)
|