Add initial implementation of SourceBufferStream

media/base/buffers.h

Index: media/base/buffers.h
diff --git a/media/base/buffers.h b/media/base/buffers.h
index f4e42642b4f240ce961d987094d59e80365c5dcb..5afe79e2d05ba4bbe88d168193f97c90c5db0d2c 100644
--- a/media/base/buffers.h
+++ b/media/base/buffers.h
@@ -73,14 +73,20 @@ class MEDIA_EXPORT Buffer : public base::RefCountedThreadSafe<Buffer> {
     duration_ = duration;
   }
 
+  // TODO(vrk): Only ChunkDemuxer and friends care about keyframes.
+  // Put keyframe stuff inside a derived class of Buffer?

[acolwell GONE FROM CHROMIUM, 2012/04/29 18:13:06]

I think you should remove this TODO. ISTM if we really were worried about this
we'd do this right now instead of temporarily putting this method here.

[vrk (LEFT CHROMIUM), 2012/05/01 22:51:13]

Done.

+  bool IsKeyframe() const { return is_keyframe_; }
+
  protected:
   friend class base::RefCountedThreadSafe<Buffer>;
   Buffer(base::TimeDelta timestamp, base::TimeDelta duration);
+  Buffer(base::TimeDelta timestamp, base::TimeDelta duration, bool is_keyframe);

[acolwell GONE FROM CHROMIUM, 2012/04/29 18:13:06]

I think we should be able to get away with just adding the parameter to the
existing constructor. It doesn't look like there are too many call sites to
update. 
You should probably make the buffer changes a different CL so we can focus on
just those changes. 

[vrk (LEFT CHROMIUM), 2012/05/01 22:51:13]

Done, CL here:
https://chromiumcodereview.appspot.com/10269022/

   virtual ~Buffer();
 
  private:
   base::TimeDelta timestamp_;
   base::TimeDelta duration_;
+  bool is_keyframe_;
 
   DISALLOW_COPY_AND_ASSIGN(Buffer);
 };