MSE: Log a warning if muxed AV media segment has no A or has no V block

media/base/stream_parser.h

 // Copyright (c) 2012 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_BASE_STREAM_PARSER_H_
 #define MEDIA_BASE_STREAM_PARSER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 88 matching lines @@
   // Return value - True indicates that the buffers are accepted.
   //                False if something was wrong with the buffers and a parsing
   //                error should be signalled.
   typedef base::Callback<bool(const BufferQueue&,
                               const BufferQueue&,
                               const TextBufferQueueMap&)> NewBuffersCB;
 
   // Signals the beginning of a new media segment.
   typedef base::Callback<void()> NewMediaSegmentCB;
 
+  // Signals the end of a media segment. Returns true if the signal is accepted.
+  // Returns false if something was wrong with segment ending and a parsing
+  // error should be signalled.
+  typedef base::Callback<bool()> EndMediaSegmentCB;
+
   // A new potentially encrypted stream has been parsed.
   // First parameter - The type of the initialization data associated with the
   //                   stream.
   // Second parameter - The initialization data associated with the stream.
   typedef base::Callback<void(EmeInitDataType, const std::vector<uint8_t>&)>
       EncryptedMediaInitDataCB;
 
   StreamParser();
   virtual ~StreamParser();
 
   // Initializes the parser with necessary callbacks. Must be called before any
   // data is passed to Parse(). |init_cb| will be called once enough data has
   // been parsed to determine the initial stream configurations, presentation
   // start time, and duration. If |ignore_text_track| is true, then no text
   // buffers should be passed later by the parser to |new_buffers_cb|.
   virtual void Init(
       const InitCB& init_cb,
       const NewConfigCB& config_cb,
       const NewBuffersCB& new_buffers_cb,
       bool ignore_text_track,
       const EncryptedMediaInitDataCB& encrypted_media_init_data_cb,
       const NewMediaSegmentCB& new_segment_cb,
-      const base::Closure& end_of_segment_cb,
+      const EndMediaSegmentCB& end_of_segment_cb,
       const scoped_refptr<MediaLog>& media_log) = 0;
 
-  // Called when a seek occurs. This flushes the current parser state
-  // and puts the parser in a state where it can receive data for the new seek
-  // point.
+  // Called during the reset parser state algorithm.  This flushes the current
+  // parser and puts the parser in a state where it can receive data.  This
+  // method does not need to invoke the EndMediaSegmentCB since the parser reset
+  // algorithm already resets the segment parsing state.
   virtual void Flush() = 0;
 
   // Called when there is new data to parse.
   //
   // Returns true if the parse succeeds.
   virtual bool Parse(const uint8_t* buf, int size) = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(StreamParser);
 };
@@ skipping 11 matching lines @@
 // subtle issues with tie-breaking. See http://crbug.com/338484.
 MEDIA_EXPORT bool MergeBufferQueues(
     const StreamParser::BufferQueue& audio_buffers,
     const StreamParser::BufferQueue& video_buffers,
     const StreamParser::TextBufferQueueMap& text_buffers,
     StreamParser::BufferQueue* merged_buffers);
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_STREAM_PARSER_H_