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

Unified Diff: media/filters/source_buffer_stream.h

Issue 10836304: Add support for config changes during playback to FFmpegVideoDecoder. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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 e67a84d0a7a56f01f905d56ddb716029d718a33e..af0c6846eb2b249415aeb82247672d335e79e54e 100644
--- a/media/filters/source_buffer_stream.h
+++ b/media/filters/source_buffer_stream.h
@@ -258,9 +258,13 @@ class MEDIA_EXPORT SourceBufferStream {
// Stores the largest distance between two adjacent buffers in this stream.
base::TimeDelta max_interbuffer_distance_;
-// The maximum amount of data in bytes the stream will keep in memory.
+ // The maximum amount of data in bytes the stream will keep in memory.
int memory_limit_;
+ // Indicates that a kConfigChanged status has been reported by GetNextBuffer()
+ // and the next GetCurrentXXXDecoderConfig() should update the current config.
Ami GONE FROM CHROMIUM 2012/08/17 15:16:59 This comment implies that the contract is that GNB
acolwell GONE FROM CHROMIUM 2012/08/20 23:10:12 That was the original intent, but I like your sugg
+ bool config_change_pending_;
+
DISALLOW_COPY_AND_ASSIGN(SourceBufferStream);
};

Powered by Google App Engine
This is Rietveld 408576698