Adding ChunkDemuxer implementation.

media/base/data_buffer.h

Index: media/base/data_buffer.h
diff --git a/media/base/data_buffer.h b/media/base/data_buffer.h
index 2e9551a03419ec184281c5d721440f822c86262e..7f11b158bd498e6ed4d0d7d220653929d77ead6a 100644
--- a/media/base/data_buffer.h
+++ b/media/base/data_buffer.h
@@ -21,6 +21,11 @@ class DataBuffer : public Buffer {
   // |buffer_size|.
   DataBuffer(uint8* buffer, size_t buffer_size);
 
+  // Takes ownership of |buffer| if |copy| is false. If |copy| is true,
+  // a copy is made of |buffer|. Assumes |buffer_size| is the actual size
+  // of |buffer|.
+  DataBuffer(uint8* buffer, size_t buffer_size, bool copy);

[scherkus (not reviewing), 2011/06/23 18:50:18]

I don't think we should have this ctor:
  a) boolean arguments aren't very clear
  b) passing false is equivalent to the first ctor
  c) there are only 2 callees

this is where Go is kind of awesome because instead of having ctors you have
static methods like "NewDataBufferWithCopiedData". anyway I think a helper
method inside your ChunkDemuxer would do the trick

note: the ctor below where you pass in a buffer size should die -- it's only
used to create EOS buffers by passing in 0, but that would be better suited with
a "EndOfStreamBuffer" subclass or something like that :)

[acolwell GONE FROM CHROMIUM, 2011/06/23 22:42:17]

Done.

+
   // Allocates buffer of size |buffer_size|. If |buffer_size| is 0, |data_| is
   // set to a NULL ptr.
   explicit DataBuffer(size_t buffer_size);