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

Unified Diff: media/filters/source_buffer_stream.h

Issue 125543002: Add plumbing and support for crossfading StreamParserBuffers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Tests! Created 6 years, 11 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/filters/source_buffer_stream.h
diff --git a/media/filters/source_buffer_stream.h b/media/filters/source_buffer_stream.h
index ff5763b42917175332ec86b84895339546bfd273..24e7d36b745aa6e389372b941436629741a63ac7 100644
--- a/media/filters/source_buffer_stream.h
+++ b/media/filters/source_buffer_stream.h
@@ -293,6 +293,11 @@ class MEDIA_EXPORT SourceBufferStream {
bool is_video() const { return video_configs_.size() > 0; }
+ // See GetNextBuffer() for additional details. The internal method hands out
+ // buffers from the |track_buffer_| and |selected_range_| without additional
+ // processing for splice frame buffers; which is handled by GetNextBuffer().
+ Status GetNextBufferInternal(scoped_refptr<StreamParserBuffer>* out_buffer);
+
// Callback used to report error strings that can help the web developer
// figure out what is wrong with the content.
LogCB log_cb_;
@@ -369,6 +374,16 @@ class MEDIA_EXPORT SourceBufferStream {
// GetCurrentXXXDecoderConfig() has been called.
bool config_change_pending_;
+ // Used by GetNextBuffer() when a buffer with fade out is returned from
+ // GetNextBufferInternal(). Will be set to the returned buffer and will be
+ // consumed after the fade out section has been exhausted.
+ scoped_refptr<StreamParserBuffer> fade_in_buffer_;
+
+ // Indicates which of the fade out preroll buffers in |fade_in_buffer_| should
+ // be handled out next.
+ size_t fade_out_preroll_index_;
+
+
DISALLOW_COPY_AND_ASSIGN(SourceBufferStream);
};

Powered by Google App Engine
This is Rietveld 408576698