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

Unified Diff: trunk/src/media/midi/midi_message_queue.h

Issue 93583002: Revert 237558 "Use MIDIMessageQueue/IsValidWebMIDIData for MIDI ..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 1 month 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
« no previous file with comments | « trunk/src/media/media.gyp ('k') | trunk/src/media/midi/midi_message_queue.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/media/midi/midi_message_queue.h
===================================================================
--- trunk/src/media/midi/midi_message_queue.h (revision 237659)
+++ trunk/src/media/midi/midi_message_queue.h (working copy)
@@ -1,72 +0,0 @@
-// Copyright 2013 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_MESSAGE_QUEUE_H_
-#define MEDIA_MIDI_MIDI_MESSAGE_QUEUE_H_
-
-#include <deque>
-#include <vector>
-
-#include "base/basictypes.h"
-#include "media/base/media_export.h"
-
-namespace media {
-
-// A simple message splitter for possibly unsafe/corrupted MIDI data stream.
-// This class allows you to:
-// - maintain fragmented MIDI message.
-// - skip any invalid data sequence.
-// - reorder MIDI messages so that "System Real Time Message", which can be
-// inserted at any point of the byte stream, is placed at the boundary of
-// complete MIDI messages.
-// - (Optional) reconstruct complete MIDI messages from data stream where
-// MIDI status byte is abbreviated (a.k.a. "running status").
-//
-// Example (pseudo message loop):
-// MIDIMessageQueue queue(true); // true to support "running status"
-// while (true) {
-// if (is_incoming_midi_data_available()) {
-// std::vector<uint8> incoming_data;
-// read_incoming_midi_data(&incoming_data)
-// queue.Add(incoming_data);
-// }
-// while (true) {
-// std::vector<uint8> next_message;
-// queue.Get(&next_message);
-// if (!next_message.empty())
-// dispatch(next_message);
-// }
-// }
-class MEDIA_EXPORT MIDIMessageQueue {
- public:
- // Initializes the queue. Set true to |allow_running_status| to enable
- // "MIDI running status" reconstruction.
- explicit MIDIMessageQueue(bool allow_running_status);
- ~MIDIMessageQueue();
-
- // Enqueues |data| to the internal buffer.
- void Add(const std::vector<uint8>& data);
- void Add(const uint8* data, size_t length);
-
- // Fills the next complete MIDI message into |message|. If |message| is
- // not empty, the data sequence falls into one of the following types of
- // MIDI message.
- // - Single "Channel Voice Message" (w/o "System Real Time Messages")
- // - Single "Channel Mode Message" (w/o "System Real Time Messages")
- // - Single "System Exclusive Message" (w/o "System Real Time Messages")
- // - Single "System Common Message" (w/o "System Real Time Messages")
- // - Single "System Real Time message"
- // |message| is empty if there is no complete MIDI message any more.
- void Get(std::vector<uint8>* message);
-
- private:
- std::deque<uint8> queue_;
- std::vector<uint8> next_message_;
- const bool allow_running_status_;
- DISALLOW_COPY_AND_ASSIGN(MIDIMessageQueue);
-};
-
-} // namespace media
-
-#endif // MEDIA_MIDI_MIDI_MESSAGE_QUEUE_H_
« no previous file with comments | « trunk/src/media/media.gyp ('k') | trunk/src/media/midi/midi_message_queue.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698