Cap the memory usage in FFMpegDemuxer.

media/base/decoder_buffer_queue.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_DECODER_BUFFER_QUEUE_H_
 #define MEDIA_BASE_DECODER_BUFFER_QUEUE_H_
 
 #include <deque>
 
 #include "base/memory/ref_counted.h"
@@ skipping 33 matching lines @@
   // Returns true if this queue is empty.
   bool IsEmpty();
 
   // Returns the duration of encoded data stored in this queue as measured by
   // the timestamps of the earliest and latest buffers, ignoring out of order
   // buffers.
   //
   // Returns zero if the queue is empty.
   base::TimeDelta Duration();
 
+  // Returns the total size of buffers inside the queue.
+  int data_size() const { return data_size_; }

[DaleCurtis, 2014/02/19 18:59:25]

I'd just make this a size_t to minimize the CHECKs you need. The style guide now
recommends size_t for "size" types.

[damienv1, 2014/02/19 19:21:10]

I agree that size_t should have been used in a first place.
However, I used "int" for consistency with DecoderBuffer::data_size which
currently returns "int".
Does it make sense to mix both data types ?
My first feeling was to use the same type and if we want to change to the
suggested size_t type, then we would change both DecoderBuffer and
DecoderBufferQueue all together.

+
  private:
   typedef std::deque<scoped_refptr<DecoderBuffer> > Queue;
   Queue queue_;
 
   // A subset of |queue_| that contains buffers that are in strictly
   // increasing timestamp order. Used to calculate Duration() while ignoring
   // out-of-order buffers.
   Queue in_order_queue_;
 
   base::TimeDelta earliest_valid_timestamp_;
 
+  // Total size in bytes of buffers in the queue.
+  int data_size_;
+
   DISALLOW_COPY_AND_ASSIGN(DecoderBufferQueue);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_DECODER_BUFFER_QUEUE_H_